arina-ielchiieva commented on a change in pull request #1345: DRILL-6494: Drill Plugins Handler URL: https://github.com/apache/drill/pull/1345#discussion_r198852690
########## File path: common/src/main/java/org/apache/drill/common/config/CommonConstants.java ########## @@ -31,4 +31,7 @@ /** Override configuration file name. (Classpath resource pathname.) */ String CONFIG_OVERRIDE_RESOURCE_PATHNAME = "drill-override.conf"; + /** Override plugins configs file name. (Classpath resource pathname.) */ + String STORAGE_PLUGINS_UPDATER_FILE = "storage-plugins.conf"; Review comment: storage-plugins-override.conf? And you can keep similar constant name: STORAGE_PLUGINS_OVERRIDE_CONF instead of "updater"... ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on 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
