GraphScope icon indicating copy to clipboard operation
GraphScope copied to clipboard

refactor(interactive): Migrating Physical Layer into the new Calcite-based Compilation Stack.

Open BingqingLyu opened this issue 1 year ago • 1 comments

Is your feature request related to a problem? Please describe.

The GIE IR layer is re-factored based on Calcite. However, the physical layer is still in the pre-IR route. Migrate it into the new Compilation Stack.

To migrate the physical layer:

  1. The following operators need to be converted to Physical Plan (in proto) Graph Operators:
  • [X] Scan
  • [X] IndexScan
  • [X] EdgeExpand
  • [X] GetV
  • [X] PathExpand
  • [X] ExpandDegree
  • [X] PhysicalEdgeExpand
  • [X] PhysicalGetV
  • [x] CommonTableScan

Relational Operators:

  • [X] Filter
  • [X] Project
  • [X] Aggregate
  • [X] Dedup
  • [X] Order
  • [X] Join
  • [x] Intersect (MultiJoin)
  • [x] Union
  • [ ] Apply

Others:

  • [ ] Sink
    • [X] SinkDefault
    • [ ] SinkVineyard
  1. The following physical rules should be implemented
  • [X] ExpandGetVFusion
    • [X] BasicExpandGetVFusion
    • [X] PathbaseExpandGetVFusion
  • [x] Repartition
  • [x] PropertyFetch
    • [x] PreCache (for edge properties)
    • [x] LateProject (for vertex properties)

BingqingLyu avatar Feb 20 '24 07:02 BingqingLyu

/cc @BingqingLyu, this issus/pr has had no activity for for a long time, could you folks help to review the status ? To suppress further notifications,

  • for issues,
    • if it is waiting for further response from the reporter/author, please help to add the label requires-further-info,
    • if you have already started working on it, please add the label work-in-progress to the issue,
    • if this issue requires further designing discussion and not in current plan, or won't be fixed, please add the label requires-further-discussion or wontfix to the issue,
  • for pull requests,
    • if you are still working on it and it is not ready for reviewing, please convert this pull request as draft PR,
    • if you have decided to hold this development on, please add the requires-further-discussion label to the pull request. Thanks!

github-actions[bot] avatar Mar 12 '24 00:03 github-actions[bot]