Making SHOW VARIABLES show boolean values as ON/OFF to match MySQL.

Author: fulghum

sql/rowexec/show.go

@@ -557,7 +557,20 @@ func (b *BaseBuilder) buildShowVariables(ctx *sql.Context, n *plan.ShowVariables
 				continue
 			}
 		}
-		rows = append(rows, sql.NewRow(k, v))
+
+		// SHOW VARIABLES displays boolean values as "ON" or "OFF".
+		if boolVal, isBoolVal := v.(int8); isBoolVal {
+			switch boolVal {
+			case 0:
+				rows = append(rows, sql.NewRow(k, "OFF"))
+			case 1:
+				rows = append(rows, sql.NewRow(k, "ON"))
+			default:
+				rows = append(rows, sql.NewRow(k, v))
+			}
+		} else {
+			rows = append(rows, sql.NewRow(k, v))
+		}
 	}
 
 	sort.Slice(rows, func(i, j int) bool {

sql/variables/system_variables.go

@@ -1286,6 +1286,24 @@ var systemVars = map[string]sql.SystemVariable{
 		//       Integrators who provide binary logging may change this default.
 		Default: int8(0),
 	},
+	"log_replica_updates": &sql.MysqlSystemVariable{
+		Name:              "log_replica_updates",
+		Scope:             sql.GetMysqlScope(sql.SystemVariableScope_Persist),
+		Dynamic:           true,
+		SetVarHintApplies: false,
+		Type:              types.NewSystemBoolType("log_replica_updates"),
+		Default:           int8(1),
+	},
+	"log_slave_updates": &sql.MysqlSystemVariable{
+		// TODO: This var should be an *alias* for log_replica_updates, but
+		//       we don't support system variable aliases yet.
+		Name:              "log_slave_updates",
+		Scope:             sql.GetMysqlScope(sql.SystemVariableScope_Persist),
+		Dynamic:           true,
+		SetVarHintApplies: false,
+		Type:              types.NewSystemBoolType("log_slave_updates"),
+		Default:           int8(1),
+	},
 	"log_error": &sql.MysqlSystemVariable{
 		Name:              "log_error",
 		Scope:             sql.GetMysqlScope(sql.SystemVariableScope_Global),
@@ -2041,7 +2059,7 @@ var systemVars = map[string]sql.SystemVariable{
 		Dynamic:           true,
 		SetVarHintApplies: false,
 		Type:              types.NewSystemUintType("query_cache_size", 0, 18446744073709551615),
-		Default:           int8(1),
+		Default:           1,
 	},
 	"query_cache_type": &sql.MysqlSystemVariable{
 		Name:              "query_cache_type",
@@ -2933,31 +2951,31 @@ var systemVars = map[string]sql.SystemVariable{
 		Dynamic:           true,
 		SetVarHintApplies: false,
 		Type:              types.NewSystemIntType("validate_password.length", 0, 2147483647, false),
-		Default:           int8(8),
+		Default:           8,
 	},
 	"validate_password.number_count": &sql.MysqlSystemVariable{
 		Name:              "validate_password.number_count",
 		Scope:             sql.GetMysqlScope(sql.SystemVariableScope_Global),
 		Dynamic:           true,
 		SetVarHintApplies: false,
 		Type:              types.NewSystemIntType("validate_password.number_count", 0, 2147483647, false),
-		Default:           int8(1),
+		Default:           1,
 	},
 	"validate_password.mixed_case_count": &sql.MysqlSystemVariable{
 		Name:              "validate_password.mixed_case_count",
 		Scope:             sql.GetMysqlScope(sql.SystemVariableScope_Global),
 		Dynamic:           true,
 		SetVarHintApplies: false,
 		Type:              types.NewSystemIntType("validate_password.mixed_case_count", 0, 2147483647, false),
-		Default:           int8(1),
+		Default:           1,
 	},
 	"validate_password.special_char_count": &sql.MysqlSystemVariable{
 		Name:              "validate_password.special_char_count",
 		Scope:             sql.GetMysqlScope(sql.SystemVariableScope_Global),
 		Dynamic:           true,
 		SetVarHintApplies: false,
 		Type:              types.NewSystemIntType("validate_password.special_char_count", 0, 2147483647, false),
-		Default:           int8(1),
+		Default:           1,
 	},
 	"validate_user_plugins": &sql.MysqlSystemVariable{
 		Name:              "validate_user_plugins",