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