Skip to content
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

feat: support multi chain inside one actor #2785

Closed
yezizp2012 opened this issue May 24, 2022 · 2 comments
Closed

feat: support multi chain inside one actor #2785

yezizp2012 opened this issue May 24, 2022 · 2 comments

Comments

@yezizp2012
Copy link
Member

I think there will still be bug. If we have one actor like:

Union
  Chain
  Chain

The fragmenter won't split two chains into different fragments due to the recursive algorithm. Currently we don't have stateless executors with multiple inputs (like lookup), but I think we will need to support multiple chain inside one actor in the future.

Originally posted by @skyzh in #2775 (comment)

@yezizp2012
Copy link
Member Author

yezizp2012 commented May 24, 2022

The reason for the current restriction on the existence of multiple chains in the actor is that the ddl status report is bound to the actor, so just change it to be bound to the chain executors or when all the chains in the actor have reported before the actor finish can be counted. This can be changed after @BugenZhao finishes this PR #2773 .

Originally posted by @yezizp2012 in #2775 (comment)

@yezizp2012
Copy link
Member Author

duplicated with #6236 .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants