-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
HBASE-25237 : 'hbase master stop' shuts down the cluster, not the master only #2705
Conversation
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
@virajjasani @saintstack please take a look! |
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.
Looks good overall, you might also want to update documentation (I think it should be: ops_mgt.adoc
)
} | ||
System.err.println( | ||
"To shutdown the master run " + | ||
"hbase-daemon.sh stop HMaster or send a kill signal to " + |
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.
Isn't it hbase-daemon.sh stop master
?
"To shutdown the master run " + | ||
"hbase-daemon.sh stop HMaster or send a kill signal to " + | ||
"the HMaster pid, " + | ||
"and to stop HBase Cluster run stop-hbase.sh or add --forceStop"); |
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.
nit: and to stop HBase Cluster either run stop-hbase.sh or add --forceStop with current command
🎊 +1 overall
This message was automatically generated. |
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
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
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.
Few comments, almost there
@@ -62,7 +62,9 @@ | |||
" --localRegionServers=<servers> " + | |||
"RegionServers to start in master process when in standalone mode.\n" + | |||
" --masters=<servers> Masters to start in this process.\n" + | |||
" --backup Master should start in backup mode"; | |||
" --backup Master should start in backup mode" + | |||
" --forceStop " + |
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.
I think forceStop
is generally referred to when we want to non-gracefully shutdown any service. Let's use flag --shutDownCluster
instead.
"To shutdown the master run " + | ||
"hbase-daemon.sh stop master or send a kill signal to " + | ||
"the HMaster pid, " + | ||
"and to stop HBase Cluster run stop-hbase.sh or add --forceStop with current command"); |
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.
Here we can specify the exact command to be used:
and to stop HBase Cluster run: "stop-hbase.sh --shutDownCluster" or "hbase master stop --shutDownCluster"
Sounds good?
🎊 +1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
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, one small nit, otherwise good to go!
usage="Usage: stop-hbase.sh can only be used for shutting down entire cluster\ | ||
to shut down (HMaster|HRegionServer) use hbase-daemon.sh stop (master|regionserver)" |
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.
Great!
"To shutdown the master run " + | ||
"hbase-daemon.sh stop master or send a kill signal to " + | ||
"the HMaster pid, " + | ||
"and to stop HBase Cluster run stop-hbase.sh or add --shutDownCluster with current command"); |
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.
nit:
and to stop HBase Cluster run: "stop-hbase.sh" or "hbase master stop --shutDownCluster"
💔 -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. |
…ter only Closes #2705 Signed-off-by: Viraj Jasani <vjasani@apache.org>
…ter only Closes #2705 Signed-off-by: Viraj Jasani <vjasani@apache.org>
…ter only Closes #2705 Signed-off-by: Viraj Jasani <vjasani@apache.org>
…ter only Closes apache#2705 Signed-off-by: Viraj Jasani <vjasani@apache.org>
…ter only Closes apache#2705 Signed-off-by: Viraj Jasani <vjasani@apache.org> (cherry picked from commit f7ae870) Change-Id: I9fca561554510759b20478f20277c3a39f214488
No description provided.