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-10066][DataProxy] Optimize the related configuration settings in the CommonConfigHolder.java file #10079

Merged
merged 1 commit into from
Apr 26, 2024
Merged

[INLONG-10066][DataProxy] Optimize the related configuration settings in the CommonConfigHolder.java file #10079

merged 1 commit into from
Apr 26, 2024

Conversation

gosonzhang
Copy link
Contributor

Fixes #10066

  1. Uniformly configure the Key to lowercase, and use "." to classify;
  2. Configure pre-reading according to different types;
  3. Optimize the acquisition of configuration values, and check whether key configurations are empty and legal;

@gosonzhang gosonzhang merged commit 60c2676 into apache:master Apr 26, 2024
9 checks passed
@gosonzhang gosonzhang deleted the master-10066 branch April 26, 2024 02:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@baomingyu baomingyu baomingyu approved these changes

@luchunliang luchunliang luchunliang approved these changes

@dockerzhang dockerzhang dockerzhang approved these changes

@fuweng11 fuweng11 Awaiting requested review from fuweng11

Assignees

@gosonzhang gosonzhang

Labels
component/dataproxy
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Improve][DataProxy] Optimize the related configuration settings in the CommonConfigHolder.java file
4 participants
@gosonzhang @baomingyu @luchunliang @dockerzhang