-
Notifications
You must be signed in to change notification settings - Fork 8.9k
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
HDFS-17657. The balancer service supports httpserver. #7242
base: trunk
Are you sure you want to change the base?
Conversation
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
e36817d
to
bf68583
Compare
looks good, but this PR might have to wait until my PR is merged first. I'm in the process of upgrading from Jersey 1.x to Jersey 2. I’ve been a bit delayed lately due to work being busy. |
@slfan1989 No problem, I'll follow up later. Thanks! |
🎊 +1 overall
This message was automatically generated. |
...ct/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/server/balancer/TestBalancerService.java
Show resolved
Hide resolved
LGTM. However, the merge time may be delayed. I will submit the final version of HADOOP-15984 next week. This PR is very important for us to upgrade JDK17, but once the httpserver is changed, it may have an impact on my changes. If there is any other code related to the HTTP server, we can submit it later. |
@slfan1989 I got, thanks~ |
Description of PR
JIRA:https://issues.apache.org/jira/browse/HDFS-17657
Background
At present, the balance service supports long process deployment, but does not support webserver services.
Implement functions
1.This PR mainly implements the basic capabilities of the webserver, and through the UI interface, it can view metrics such as the amount of data currently being migrated by the balancer process, how much data the cluster still needs to migrate to reach balanced state, etc.
2.Reuse the basic functions of HttpServer and HTML, support view conf, metrics, thread dump, etc.
Figures
How was this patch tested?
Add UT