BukrosSzabolcs commented on a change in pull request #592: HBASE-22982: region
server suspend/resume and graceful rolling restart actions
URL: https://github.com/apache/hbase/pull/592#discussion_r328118752
##########
File path:
hbase-it/src/test/java/org/apache/hadoop/hbase/chaos/actions/RestartActionBaseAction.java
##########
@@ -50,6 +50,23 @@ void restartMaster(ServerName server, long sleepTime)
throws IOException {
startMaster(server);
}
+ /**
+ * Stop and then restart the region server instaedof killing it.
+ * @param server hostname to restart the regionserver on
+ * @param sleepTime number of milliseconds between stop and restart
+ * @throws IOException if something goes wrong
+ */
+ void gracefulRestartRs(ServerName server, long sleepTime) throws IOException
{
+ sleepTime = Math.max(sleepTime, 1000);
+ // Don't try the stop if we're stopping already
+ if (context.isStopping()) {
+ return;
+ }
+ stopRs(server);
Review comment:
I'll add Stopping/Starting logs to the other methods too.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
