-
Notifications
You must be signed in to change notification settings - Fork 929
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.
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
Add: user define config #1640
Add: user define config #1640
Conversation
2c99e37
to
35630bd
Compare
Codecov Report
@@ Coverage Diff @@
## 3.0 #1640 +/- ##
==========================================
+ Coverage 41.05% 41.35% +0.29%
==========================================
Files 253 254 +1
Lines 14501 14541 +40
==========================================
+ Hits 5953 6013 +60
+ Misses 7859 7835 -24
- Partials 689 693 +4
Continue to review full report at Codecov.
|
👍 |
group: test | ||
address: 127.0.0.1:2181 | ||
custom: | ||
config-map: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这里缩进是不是有问题? @A-Wanderer
事实证明是这个问题,我认为这个pr可以merge了。相关rootConfig问题麻烦记录个issue。 |
What this PR does:
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?: