ctubbsii opened a new issue, #58: URL: https://github.com/apache/accumulo-access/issues/58
Currently, Authorizations is implemented as an unordered set (`Set.copyOf(authorizations)`). It also has some unnecessary copying (`Authorizations.of(Collection<String>)` results in two separate calls to `Set.copyOf`, one of which is redundant). When calling `asSet` to view the result, the returned set could be in any order. It would be nice to make some improvements, so that a particular set of strings had a canonical view. This may also have implications for hashCode and equals if the implementation stores them in different orderings, but I'm not certain about that. It's also just nice to have a sorted view of the authorizations when calling `asSet` or `toString`. -- 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]
