ctubbsii opened a new pull request, #5067: URL: https://github.com/apache/accumulo/pull/5067
* Add a check to make sure that if the name exists in the mapping, then it results in a namespace exists exception, rather than only do that if the ID exists in the mapping (this fixes the NamespaceIT failure seen after #4996 was merged) * Make NamespaceMapping.put() idempotent by checking the current mapping against the desired mapping (this preserves existing behavior at this point in the code, which was previously being handled by the Utils method that was deleted) Trivial changes: * Add some stricter checks for parameters to NamespaceMapping methods (final and require non-null) * Avoid NPE problems by comparing the known non-null string on the left side of the .equals() * Make name and inline comments consistent between rename and put methods -- 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]
