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

[Improvement] [tez] Optimize tez client delivery configuration #983

Closed
3 tasks done
zhengchenyu opened this issue Jun 30, 2023 · 0 comments
Closed
3 tasks done

[Improvement] [tez] Optimize tez client delivery configuration #983

zhengchenyu opened this issue Jun 30, 2023 · 0 comments

Comments

@zhengchenyu
Copy link
Collaborator

Code of Conduct

Search before asking

  • I have searched in the issues and found no similar issues.

What would you like to be improved?

Three improvement about configuration will be done in this issue.

1 For now, tez client use rss_conf.xml to delivery configuration. As #966 is applied, we can delivery configuration by edge conf.

2 delivery dynamic configuration from shuffle server, then override the tez client configuration.

3 delivery configuration from client side.

How should we improve?

No response

Are you willing to submit PR?

  • Yes I am willing to submit a PR!
zhengchenyu added a commit to zhengchenyu/incubator-uniffle that referenced this issue Jun 30, 2023
zhengchenyu added a commit to zhengchenyu/incubator-uniffle that referenced this issue Jul 3, 2023
zhengchenyu added a commit to zhengchenyu/incubator-uniffle that referenced this issue Jul 3, 2023
jerqi pushed a commit that referenced this issue Jul 4, 2023
…985)

### What changes were proposed in this pull request?

Three improvement about configuration will be done in this issue.

- 1 For now, tez client use rss_conf.xml to delivery configuration. As #966 is applied, we can delivery configuration by edge conf.
- 2 delivery dynamic configuration from coordinator, then override the tez client configuration.
- 3 delivery configuration from client side.

### Why are the changes needed?

- 1. rss_conf.xml is unnecessary.
- 2. dynamic configuration from coordinator are not applied.
- 3. config in client side can not delivery to input/ouput.

### How was this patch tested?

integration test, unit test, test in yarn cluster, test in tez local mode.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant