-
Notifications
You must be signed in to change notification settings - Fork 5k
[Improvement] Optimize server startup log #15362
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
Conversation
9d2aabd to
401b9d0
Compare
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## dev #15362 +/- ##
============================================
+ Coverage 38.10% 38.11% +0.01%
Complexity 4697 4697
============================================
Files 1301 1300 -1
Lines 44800 44775 -25
Branches 4800 4800
============================================
- Hits 17070 17065 -5
+ Misses 25877 25858 -19
+ Partials 1853 1852 -1 ☔ View full report in Codecov by Sentry. |
401b9d0 to
ccbfc38
Compare
...uler-master/src/main/java/org/apache/dolphinscheduler/server/master/config/MasterConfig.java
Outdated
Show resolved
Hide resolved
...eduler-task-api/src/main/java/org/apache/dolphinscheduler/plugin/task/api/TaskConstants.java
Show resolved
Hide resolved
|
Please add some pr description. |
Done |
ccbfc38 to
4c85a57
Compare
caishunfeng
left a comment
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.
LGTM
4c85a57 to
8716eb9
Compare
8716eb9 to
bac32c6
Compare
|

Purpose of the pull request
config format,Rpc server,log patten of class.Brief change log
Verify this pull request
This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(or)
If your pull request contain incompatible change, you should also add it to
docs/docs/en/guide/upgrede/incompatible.md