EdColeman opened a new issue, #2633: URL: https://github.com/apache/accumulo/issues/2633
As a follow-on to single node property storage in ZooKeeper (issue #1454) The code contains classes NamespacePropUtil and TablePropUtil that have a lot of overlap - one issue is that the setProperties() methods handle invalid properties differently. On setting an invalid property: - on a namespace, the method is declared void and throws an exception - on a table, the method is declared to returns a boolean and returns false of the property is invalid. If one method was chosen, then duplicate code could be removed by combining the functionality into a single class used by namespace and tables. -- 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]
