BiteTheDDDDt commented on code in PR #9243:
URL: https://github.com/apache/incubator-doris/pull/9243#discussion_r858638857
##########
gensrc/thrift/PaloInternalService.thrift:
##########
@@ -274,8 +274,18 @@ struct TTxnParams {
10: optional double max_filter_ratio
}
-// ExecPlanFragment
+// Definition of global dict, global dict is used to accelerate query
performance of low cardinality data
+struct TColumnDict {
+ 1: optional Types.TPrimitiveType type
+ 2: list<string> str_dict // map one string to a integer, using offset as id
+}
+struct TGlobalDict {
+ 1: optional map<i32, TColumnDict> dicts, // map dict_id to column dict
Review Comment:
Do we need a type as big as i32 here?
--
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: commits-unsubscr...@doris.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org
