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
[GAIA Compiler] Build algebra layer structures for graph operators #2438
Conversation
975a716
to
97dfa1f
Compare
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #2438 +/- ##
=======================================
Coverage 39.93% 39.93%
=======================================
Files 88 88
Lines 9771 9771
=======================================
Hits 3902 3902
Misses 5869 5869 Continue to review full report at Codecov.
|
GraphOpt.GetV.END, | ||
new LabelConfig(false).addLabel("person"))) | ||
.range(1, 3) | ||
.pathOpt(PathOpt.Simple) |
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.
PathOpt和ResultOpt可否也统一到GraphOpt.PathExpand.Simple和 GraphOpt.PathExpand.AllV ?
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.
PathOpt和ResultOpt还是要区分开,GraphOpt里加了PathExpandPath和PathExpandResult
public TableConfig(List<RelOptTable> tables) { | ||
this.tables = ObjectUtils.requireNonEmpty(tables); | ||
; | ||
this.isAll = false; |
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.
Please fix code format issue.
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
What do these changes do?
com.alibaba.graphscope.common.ir.schema
)com.alibaba.graphscope.common.ir.type
)Related issue number
Fixes