Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix Logic error in confProp parameter processing #1477

Merged
merged 1 commit into from
Jan 10, 2023

Conversation

ChangjunZhang
Copy link
Contributor

@ChangjunZhang ChangjunZhang commented Jan 9, 2023

[Bug- #1473 ][chunjun-clients] fix Logic error in confProp parameter processing

Purpose of this pull request

Which issue you fix

Fixes # (issue).

Checklist:

  • I have executed the 'mvn spotless:apply' command to format my code.
  • I have a meaningful commit message (including the issue id, the template of commit message is '[label-type-#issue-id][fixed-module] a meaningful commit message.')
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • I have added tests that prove my fix is effective or that my feature works.
  • New and existing unit tests pass locally with my changes.
  • I have checked my code and corrected any misspellings.
  • My commit is only one. (If there are multiple commits, you can use 'git squash' to compress multiple commits into one.)

[Bug- DTStack#1473 ][chunjun-clients] fix Logic error in confProp parameter processing
@FlechazoW FlechazoW linked an issue Jan 10, 2023 that may be closed by this pull request
3 tasks
@FlechazoW FlechazoW merged commit df98c32 into DTStack:master Jan 10, 2023
ll076110 pushed a commit that referenced this pull request Mar 3, 2023
…processing.(#1477)

[Bug- #1473 ][chunjun-clients] fix Logic error in confProp parameter processing

(cherry picked from commit df98c32)
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] [chunjun-clients] Logic error in confProp parameter processing
2 participants