morrySnow commented on code in PR #10786:
URL: https://github.com/apache/doris/pull/10786#discussion_r918895101
##########
fe/fe-core/src/main/java/org/apache/doris/nereids/analyzer/UnboundRelation.java:
##########
@@ -72,6 +85,16 @@ public LogicalProperties computeLogicalProperties(Plan...
inputs) {
return new UnboundLogicalProperties();
}
+ @Override
+ public Plan withGroupExpression(Optional<GroupExpression> groupExpression)
{
+ return new UnboundRelation(nameParts, groupExpression,
Optional.of(logicalProperties));
+ }
+
+ @Override
+ public Plan withLogicalProperties(Optional<LogicalProperties>
logicalProperties) {
+ return new UnboundRelation(nameParts, Optional.empty(),
logicalProperties);
Review Comment:
groupexpression should use original one？
##########
fe/fe-core/src/main/java/org/apache/doris/nereids/memo/GroupExpression.java:
##########
@@ -89,8 +89,8 @@ public void setParent(Group parent) {
this.parent = parent;
}
- public Operator getOperator() {
- return op;
+ public Plan getPlan() {
+ return plan;
Review Comment:
we need to set group's logical properties to plan
##########
fe/fe-core/src/main/java/org/apache/doris/analysis/DateLiteral.java:
##########
@@ -158,7 +158,8 @@ private enum DateLiteralType {
DATEV2(3);
private final int value;
- private DateLiteralType(int value) {
+
+ DateLiteralType(int value) {
Review Comment:
why do this?
--
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
