saintstack commented on a change in pull request #2731:
URL: https://github.com/apache/hbase/pull/2731#discussion_r534367135
##########
File path:
hbase-server/src/test/java/org/apache/hadoop/hbase/replication/regionserver/TestRefreshRecoveredReplication.java
##########
@@ -75,6 +80,7 @@
@BeforeClass
public static void setUpBeforeClass() throws Exception {
+ // NUM_SLAVES1 is presumed 2 in below.
Review comment:
The NUM_SLAVES1 comes from base class.
This class and the subclasses need major refactoring. I avoid it in here
trying to keep change minimal.
----------------------------------------------------------------
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:
[email protected]
