hanahmily commented on code in PR #155:
URL:
https://github.com/apache/skywalking-banyandb/pull/155#discussion_r946847339
##########
api/proto/banyandb/model/v1/query.proto:
##########
@@ -68,8 +72,23 @@ message Condition {
// tag_families are indexed.
message Criteria {
- string tag_family_name = 1;
- repeated model.v1.Condition conditions = 2;
+ repeated LogicalExpression les = 1;
+ repeated Criteria inners = 2;
+}
+
+// LogicalExpression supports logical operation
+message LogicalExpression {
+ enum LogicalOp {
+ LOGICAL_OP_UNSPECIFIED = 0;
+ LOGICAL_OP_AND = 1;
+ LOGICAL_OP_OR = 2;
+ }
+ // op is a logial operation
+ LogicalOp op = 1;
+ // tag_family_name indicates the tag family to which the tag name in the
condition belongs.
+ string tag_family_name = 2;
+ // condition is a binary operation
+ model.v1.Condition condition = 3;
}
Review Comment:
Thanks for pointing out the missing operator between inner and outer. I will
polish the API a bit.
--
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]
