-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
[FIX] Could not find or load main class org.apache.zookeeper.ZooKeeperMain #10535
Conversation
I think this is a serious issue of pseudo-cluster installation. So I put this to |
Codecov Report
@@ Coverage Diff @@
## dev #10535 +/- ##
============================================
- Coverage 40.91% 40.90% -0.02%
+ Complexity 4874 4871 -3
============================================
Files 893 893
Lines 36153 36153
Branches 4002 4002
============================================
- Hits 14793 14788 -5
- Misses 19903 19907 +4
- Partials 1457 1458 +1
Continue to review full report at Codecov.
|
Kudos, SonarCloud Quality Gate passed! |
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. Thanks.
Thanks for your contribution. This is a good beginning. Considering that it's your first contribution, I think we can get deep communication, you can contact me through mail or add WeChat(SbloodyS), when mail or added, please tell me who you are. I think I can help to familiar with the Dolphinscheduler if you meet with problems. @Desperado2 |
Agrees with that. I will cherry-pick later. |
…rMain (apache#10535) * Update remove-zk-node.sh * Update remove-zk-node.sh * Update remove-zk-node.sh * Update remove-zk-node.sh
Purpose of the pull request
fix: #10029
Brief change log
update remove-zk-node.sh
Verify this pull request
This pull request is code cleanup without any test coverage.