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.

Sign up for GitHub

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

Jump to bottom

[INLONG-10067][DataProxy] Simplify the configuration and acquisition of the Manager address #10068

Merged
merged 3 commits into from
Apr 25, 2024
Merged

[INLONG-10067][DataProxy] Simplify the configuration and acquisition of the Manager address #10068

merged 3 commits into from
Apr 25, 2024

Conversation

gosonzhang
Copy link
Contributor

Fixes #10067

  1. Delete the manager.type configuration, directly obtain, parse, and check the manager.hosts configuration key;
  2. Adjust the configuration key name related to Manager and add compatibility check;
  3. Adjust the logic of obtaining the Manager in ConfigManager and change it to obtain the Manager address;
  4. Delete manager.type related files DefaultManagerIpListParser.java, IManagerIpListParser.java

@gosonzhang gosonzhang merged commit 5a0ae62 into apache:master Apr 25, 2024
8 checks passed
@gosonzhang gosonzhang deleted the master-10067 branch April 25, 2024 04:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aloyszhang aloyszhang aloyszhang approved these changes

@fuweng11 fuweng11 fuweng11 approved these changes

Assignees

@gosonzhang gosonzhang

Labels
component/dataproxy
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Improve][DataProxy] Simplify the configuration and acquisition of the Manager address
3 participants
@gosonzhang @aloyszhang @fuweng11