apache / helix

Mirror of Apache Helix
Apache License 2.0
457 stars 218 forks source link

Fix config for metaclient #2807

Closed xyuanlu closed 1 month ago

xyuanlu commented 1 month ago

Issues

2808

Description

Config for leader election client should cover all params in metaclient config.

Tests

(If CI test fails due to known issue, please specify the issue and test PR locally. Then copy & paste the result of "mvn test" to here.)

Changes that Break Backward Compatibility (Optional)

(Consider including all behavior changes for public methods or API. Also include these changes in merge description so that other developers are aware of these changes. This allows them to make relevant code changes in feature branches accounting for the new method/API behavior.)

Documentation (Optional)

(Link the GitHub wiki you added)

Commits

Code Quality

xyuanlu commented 1 month ago

this change is ready to be merged. approved by @junkaixue

commit msg: Fix config for metaclient leader election client.