[GitHub] [accumulo] EdColeman commented on a diff in pull request #3073: Remove Connector and Instance legacy client APIs

[GitBox <gi...@apache.org>]

EdColeman commented on code in PR #3073:
URL: https://github.com/apache/accumulo/pull/3073#discussion_r1017237125


##########
core/src/main/java/org/apache/accumulo/core/client/AccumuloClient.java:
##########
@@ -52,7 +52,7 @@
  *
  * <p>
  * If migrating code from Connector to AccumuloClient an important difference to consider is that
- * AccumuloClient is closable and Connector is not. Connector uses static resources and therefore
+ * AccumuloClient is closable and Connector was not. Connector uses static resources and therefore
  * creating them is cheap. AccumuloClient attempts to clean up resources on close, so constantly

Review Comment:
   The Connector used static resources and was cheaper to create that the AccumuloClient.  However, ...



-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org