[GitHub] [incubator-seatunnel] EricJoy2048 commented on pull request #2316: [Engine][PhysicalPlan] Add code from LogicalDag to PhysicalPlan

[GitBox <gi...@apache.org>]

EricJoy2048 commented on PR #2316:
URL: https://github.com/apache/incubator-seatunnel/pull/2316#issuecomment-1200837468

   I see you use `Vertex` in PhysicalPlan. I don't think the vertices of logical plan and physical plan are the same. In physical plan physical vertices need to express y a vertex that can be executed or further converted into a task. If the value of the parallelism attribute of a logical vertex is 4, there should be at least 4 physical vertices in the corresponding physical plan, and each physical vertex should contain an index attribute to express the index of the physical vertex.


-- 
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-unsubscribe@seatunnel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org