ctubbsii commented on a change in pull request #412: #408 Removed use of
deprecated ClientConfiguration
URL: https://github.com/apache/accumulo/pull/412#discussion_r178662386
##########
File path:
core/src/main/java/org/apache/accumulo/core/client/ConnectionInfo.java
##########
@@ -35,7 +36,7 @@
/**
* @return Zookeeper connection information for Accumulo instance
*/
- String getZookeepers();
+ String getZooKeepers();
Review comment:
If it's public API, we should follow upstream naming conventions.
Internally, it doesn't really matter, other than as best practice for
developing the habit of using the trademark properly, and perhaps also for
consistency.
----------------------------------------------------------------
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
