Fix the conversion between SqlTypeId and string。 (#1635)

linmagit · web-flow · commit c7e6949985c0 · 2021-07-28T21:53:21.000+08:00
diff --git a/src/execution/sql/sql.cpp b/src/execution/sql/sql.cpp
@@ -279,20 +279,20 @@ std::string SqlTypeIdToString(SqlTypeId type) {
     case SqlTypeId::Timestamp:
       return "Timestamp";
     case SqlTypeId::Varchar:
-      return "VarChar";
+      return "Varchar";
     case SqlTypeId::Varbinary:
-      return "VarBinary";
+      return "Varbinary";
     default:
       // All cases handled
       UNREACHABLE("Impossible type");
   }
 }
 
 SqlTypeId SqlTypeIdFromString(const std::string &type_string) {
-  if (type_string == "INVALID") {
+  if (type_string == "Invalid") {
     return SqlTypeId::Invalid;
   }
-  if (type_string == "BOOLEAN") {
+  if (type_string == "Boolean") {
     return SqlTypeId::Boolean;
   }
   if (type_string == "TinyInt") {

Original file line number	Diff line number	Diff line change
`@@ -279,20 +279,20 @@ std::string SqlTypeIdToString(SqlTypeId type) {`
`279`	`279`	`case SqlTypeId::Timestamp:`
`280`	`280`	`return "Timestamp";`
`281`	`281`	`case SqlTypeId::Varchar:`
`282`		`- return "VarChar";`
	`282`	`+ return "Varchar";`
`283`	`283`	`case SqlTypeId::Varbinary:`
`284`		`- return "VarBinary";`
	`284`	`+ return "Varbinary";`
`285`	`285`	`default:`
`286`	`286`	`// All cases handled`
`287`	`287`	`UNREACHABLE("Impossible type");`
`288`	`288`	`}`
`289`	`289`	`}`
`290`	`290`
`291`	`291`	`SqlTypeId SqlTypeIdFromString(const std::string &type_string) {`
`292`		`- if (type_string == "INVALID") {`
	`292`	`+ if (type_string == "Invalid") {`
`293`	`293`	`return SqlTypeId::Invalid;`
`294`	`294`	`}`
`295`		`- if (type_string == "BOOLEAN") {`
	`295`	`+ if (type_string == "Boolean") {`
`296`	`296`	`return SqlTypeId::Boolean;`
`297`	`297`	`}`
`298`	`298`	`if (type_string == "TinyInt") {`