xiaochen-zhou commented on code in PR #4123:
URL: https://github.com/apache/calcite/pull/4123#discussion_r1903179206


##########
core/src/main/java/org/apache/calcite/sql/dialect/StarRocksSqlDialect.java:
##########
@@ -41,9 +43,32 @@
  */
 public class StarRocksSqlDialect extends MysqlSqlDialect {
 
+  public static final RelDataTypeSystem STARROCKS_TYPE_SYSTEM =
+      new RelDataTypeSystemImpl() {
+        @Override public int getMaxPrecision(SqlTypeName typeName) {
+          switch (typeName) {
+          case CHAR:
+            return 255;
+          case VARCHAR:
+            return 65533;
+          case VARBINARY:

Review Comment:
   > Is `VARBINARY` different from what in MySQL? I noticed this is the only 
difference, maybe MySQL and StarRocks can share this type system to get max 
precision.
   
   I thought it would be the same as MySQL, but what’s strange is that the 
StarRocks official website lists the maximum as 65,533  
   
![image](https://github.com/user-attachments/assets/5dfcbc21-f4f6-4ced-aebf-ba73f0210aef)
   



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to