incubator-uniffle icon indicating copy to clipboard operation
incubator-uniffle copied to clipboard

[FEATURE] Dedicated retry times on request assignment when partition reassign

Open zuston opened this issue 6 months ago • 2 comments

Code of Conduct

Search before asking

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

Describe the feature

Currently, the default assignment timeout interval and retry count are used, which is meaningless since gRPC will already have timed out and interrupted before the assignment timeout takes effect.

Motivation

No response

Describe the solution

No response

Additional context

No response

Are you willing to submit PR?

  • [ ] Yes I am willing to submit a PR!

zuston avatar Aug 28 '25 02:08 zuston

Hi @zuston

Do we need to add a new configuration option for a retry logic with backoff and assign it to GrpcClient?

https://github.com/apache/uniffle/blob/1e48bc673d1c0ee41f889a0de6192b0fab131467/common/src/main/java/org/apache/uniffle/common/config/RssClientConf.java

cchung100m avatar Sep 06 '25 06:09 cchung100m

Hi @zuston

Do we need to add a new configuration option for a retry logic with backoff and assign it to GrpcClient?

https://github.com/apache/uniffle/blob/1e48bc673d1c0ee41f889a0de6192b0fab131467/common/src/main/java/org/apache/uniffle/common/config/RssClientConf.java

Yes. This option is dedicated for the partition reassign mechanism on requesting assignments

zuston avatar Sep 09 '25 02:09 zuston