capistrant commented on code in PR #20063: URL: https://github.com/apache/druid/pull/20063#discussion_r3814696056
########## server/src/main/java/org/apache/druid/metadata/MetadataRuleManager.java: ########## @@ -41,6 +42,14 @@ public interface MetadataRuleManager List<Rule> getRulesWithDefault(String dataSource); Review Comment: got rid of them in favor of forcing everything through the snapshot. I now create an atomic reference of the snapshot on each poll instead of creating it in the getter as the initial commit did -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
