Tanya-W commented on code in PR #13035: URL: https://github.com/apache/doris/pull/13035#discussion_r1057171538
########## gensrc/thrift/PaloInternalService.thrift: ########## @@ -513,13 +513,21 @@ struct TFetchDataResult { 4: optional Status.TStatus status } +enum TCompoundType { + UNKNOWN = 0, + AND = 1, + OR = 2, + NOT = 3, +} + struct TCondition { 1: required string column_name 2: required string condition_op 3: required list<string> condition_values // In delete condition, the different column may have same column name, need // using unique id to distinguish them 4: optional i32 column_unique_id + 5: optional TCompoundType compound_type = TCompoundType.UNKNOWN Review Comment: condition_op is condition operator, such as '=' '>' '<' and so on, compound_type is combination type between multiple conditions, such as 'and' 'or' 'not', but compound_type is unused, has been removed. -- 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