milleruntime commented on PR #2777: URL: https://github.com/apache/accumulo/pull/2777#issuecomment-1156341257
> Can't you just make ByteSequence serializable? ArrayList and HashSet already are. It looks like we may not have to. By changing the private types from interfaces to classes, the warnings went away: ```diff - private Set<ByteSequence> auths = new HashSet<>(); - private List<byte[]> authsList = new ArrayList<>(); // sorted order + private final HashSet<ByteSequence> auths = new HashSet<>(); + private final ArrayList<byte[]> authsList = new ArrayList<>(); // sorted order ``` -- 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]
