-
Notifications
You must be signed in to change notification settings - Fork 529
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
[INLONG-2229] Support pulsar anthentication in Manager and Sort #2249
Conversation
…nto INLONG-2229 "merge master"
inlong-manager/manager-service/src/test/resources/application-test.properties
Show resolved
Hide resolved
Codecov Report
@@ Coverage Diff @@
## master #2249 +/- ##
============================================
- Coverage 12.56% 12.54% -0.02%
- Complexity 1210 1219 +9
============================================
Files 417 420 +3
Lines 35914 36208 +294
Branches 5635 5674 +39
============================================
+ Hits 4512 4543 +31
- Misses 30599 30859 +260
- Partials 803 806 +3
Continue to review full report at Codecov.
|
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.
+1
Title Name: [INLONG-XYZ][component] Title of the pull request
where XYZ should be replaced by the actual issue number.
Fixes #2229
Motivation
Explain here the context, and why you're making that change. What is the problem you're trying to solve.
Modifications
Describe the modifications you've done.
Verifying this change
(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)
Documentation