Re: [PR] Sort set in Authorizations.toString() and add immutability test [accumulo-access]

["dlmarion (via GitHub)" <gi...@apache.org>]

dlmarion commented on code in PR #65:
URL: https://github.com/apache/accumulo-access/pull/65#discussion_r2636346138


##########
src/main/java/org/apache/accumulo/access/Authorizations.java:
##########
@@ -58,9 +59,12 @@ public int hashCode() {
     return authorizations.hashCode();
   }
 
+  /**
+   * @return a String containing the sorted, comma-separated set of authorizations
+   */
   @Override
   public String toString() {
-    return authorizations.toString();
+    return new TreeSet<>(authorizations).toString();

Review Comment:
   @ctubbsii - are you saying we can close this and the associated issue?



-- 
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