paul-rogers commented on issue #1652: DRILL-7046: Support for loading and parsing new RM config file URL: https://github.com/apache/drill/pull/1652#issuecomment-466706140 Quick question. Why do we need a separate RM config file? Why can't this just be part of the existing Drill config file? By having a separate file, it just makes yet another detail for users to understand and distribute. I also wonder if this would suggest to future developers that each new feature should have its own config file? As a counter-example, the earlier "large/small queue" RM system hosted it's config settings within the existing Drill config file? Perhaps an intro comment for this commit could explain the reasoning?
---------------------------------------------------------------- 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: [email protected] With regards, Apache Git Services
