DomGarguilo opened a new pull request, #5893: URL: https://github.com/apache/accumulo/pull/5893
ClusterConfigParser and ResourceGroupId were doing very similar validation checks and exception throwing, so I consolidated that into a single helper method. This let me eliminate the ClusterConfigParser.validateGroupNames() method entirely and just use ResourceGroupId.validateGroupName() directly in the existing parsing loop within ClusterConfigParser. I also moved the associated test case from ClusterConfigParserTest into ResourceGroupIdTest where it fits better -- 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]
