gortiz commented on code in PR #13733: URL: https://github.com/apache/pinot/pull/13733#discussion_r1750317381
########## pinot-common/src/main/proto/plan.proto: ########## @@ -193,3 +194,29 @@ message WindowNode { int32 upperBound = 6; repeated Literal constants = 7; } + +message ExplainNode { + string type = 1; + map<string, AttributeValue> attributes = 2; + + message AttributeValue { + MergeType mergeType = 1; + + oneof value { + string string = 2; + int64 long = 3; + bool bool = 4; + string json = 5; + }; + + enum MergeType { + // Longs will be added, other types will behave like IDEMPOTENT. + DEFAULT = 0; + // When being merge, values must be the same. Otherwise cannot be merged. + IDEMPOTENT = 1; + // When being merge, if values are different, the value can be ignored. Review Comment: Done -- 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...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org