-
Notifications
You must be signed in to change notification settings - Fork 430
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[GIE/Physical] Introduce new physical operators (in proto) and physical plan builder #2411
Conversation
a4e9cdc
to
9920b6c
Compare
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #2411 +/- ##
===========================================
- Coverage 72.01% 39.93% -32.09%
===========================================
Files 88 88
Lines 10120 9771 -349
===========================================
- Hits 7288 3902 -3386
- Misses 2832 5869 +3037
Continue to review full report at Codecov.
|
e80bf60
to
3c70789
Compare
} | ||
|
||
// A dummy node to delegate a source opr for multiple scan cases. | ||
message Dummy {} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Name it as RootScan
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
import "expr.proto"; | ||
import "schema.proto"; | ||
import "type.proto"; | ||
import "algebra.proto"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Suggest redefine those operators even they are the same as the logical ones.
db120c1
to
fac34e5
Compare
PathExpand path = 32; | ||
} | ||
} | ||
message ColumnType { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这个建议叫MetaData。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
5e4a791
to
d6cc5a4
Compare
8cd330e
to
dd6c2e0
Compare
What do these changes do?
Related issue number
#2417