eldenmoon commented on code in PR #69222:
URL: https://github.com/apache/doris/pull/69222#discussion_r4522631066
##########
gensrc/thrift/Types.thrift:
##########
@@ -136,6 +136,15 @@ enum TInvertedIndexFileStorageFormat {
SNII = 4 // SNII native inverted index storage format
}
+// One field of a VARIANT Schema Template, in declaration order, for the BE's
JSON parser: a number on a
+// path whose first matching field is DECIMAL (or ARRAY<DECIMAL>) is parsed
from its text instead of
+// through a binary64. The whole template rides so the BE matches a path the
way the shredder does.
+struct TVariantTemplatePath {
+ 1: required string pattern
+ 2: required bool is_glob
+ 3: required bool is_decimal
Review Comment:
how about using real type here? instead of a boolean flag?
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
