[GitHub] [druid] maytasm commented on a change in pull request #10054: Add safeguard to make sure new Rules added are aware of Rule usage in loadstatus API

2020-06-19 Thread GitBox
maytasm commented on a change in pull request #10054: URL: https://github.com/apache/druid/pull/10054#discussion_r442745753 ## File path: server/src/main/java/org/apache/druid/server/coordinator/rules/Rule.java ## @@ -51,6 +51,14 @@ boolean appliesTo(Interval interval, Da

[GitHub] [druid] maytasm commented on a change in pull request #10054: Add safeguard to make sure new Rules added are aware of Rule usage in loadstatus API

2020-06-18 Thread GitBox
maytasm commented on a change in pull request #10054: URL: https://github.com/apache/druid/pull/10054#discussion_r442608656 ## File path: server/src/main/java/org/apache/druid/server/coordinator/rules/Rule.java ## @@ -51,6 +51,14 @@ boolean appliesTo(Interval interval, Da