cache session charset

Author: max-hoffman

sql/base_session.go

@@ -47,6 +47,7 @@ type BaseSession struct {
 	lastQueryInfo    map[string]*atomic.Value
 	tx               Transaction
 	ignoreAutocommit bool
+	charset          CharacterSetID
 
 	// When the MySQL database updates any tables related to privileges, it increments its counter. We then update our
 	// privilege set if our counter doesn't equal the database's counter.
@@ -263,15 +264,18 @@ func (s *BaseSession) GetCharacterSet() CharacterSetID {
 
 // GetCharacterSetResults returns the result character set for this session (defined by the system variable `character_set_results`).
 func (s *BaseSession) GetCharacterSetResults() CharacterSetID {
-	sysVar, _ := s.systemVars[characterSetResultsSysVarName]
-	if sysVar.Val == nil {
-		return CharacterSet_Unspecified
-	}
-	charSet, err := ParseCharacterSet(sysVar.Val.(string))
-	if err != nil {
-		panic(err) // shouldn't happen
+	if s.charset == CharacterSet_Unspecified {
+		sysVar, _ := s.systemVars[characterSetResultsSysVarName]
+		if sysVar.Val == nil {
+			return CharacterSet_Unspecified
+		}
+		var err error
+		s.charset, err = ParseCharacterSet(sysVar.Val.(string))
+		if err != nil {
+			panic(err) // shouldn't happen
+		}
 	}
-	return charSet
+	return s.charset
 }
 
 // GetCollation returns the collation for this session (defined by the system variable `collation_connection`).