wchevreuil commented on a change in pull request #1217: HBASE-23891: Add an 
option to Actions to filter out meta RS
URL: https://github.com/apache/hbase/pull/1217#discussion_r388386365
 
 

 ##########
 File path: 
hbase-it/src/test/java/org/apache/hadoop/hbase/chaos/factories/MobNoKillMonkeyFactory.java
 ##########
 @@ -75,7 +75,7 @@
 
     Action[] actions4 = new Action[] { new DumpClusterStatusAction() };
 
-    return new PolicyBasedChaosMonkey(util,
+    return new PolicyBasedChaosMonkey(properties, util,
 
 Review comment:
   Do we really need to specify extra properties when creating policy for all 
these factories? Otherwise, if this is just duo to the change in 
PolicyBasedChaosMonkey constructor, it would benefit from overloading the 
constructor there.

----------------------------------------------------------------
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

Reply via email to