----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20702/#review41460 -----------------------------------------------------------
core/src/main/java/org/apache/accumulo/core/security/Credentials.java <https://reviews.apache.org/r/20702/#comment74892> Nit: add @param and @return server/base/src/main/java/org/apache/accumulo/server/security/SecurityOperation.java <https://reviews.apache.org/r/20702/#comment74893> This check for whether the token class is valid according to the current authenticator appears to be gone. (Maybe it's not needed and/or this has already been discussed.) - Bill Havanki On April 25, 2014, 3:07 a.m., Sean Busbey wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/20702/ > ----------------------------------------------------------- > > (Updated April 25, 2014, 3:07 a.m.) > > > Review request for accumulo. > > > Bugs: ACCUMULO-2733 > https://issues.apache.org/jira/browse/ACCUMULO-2733 > > > Repository: accumulo > > > Description > ------- > > adds fromThrift to Credentials, consolidates existing one-off > deserializations. > > If ACCUMULO-2726 goes in first, I'll refactor this patch to just do the > consolidation. > > > Diffs > ----- > > core/src/main/java/org/apache/accumulo/core/security/Credentials.java > 5afc6e8 > core/src/test/java/org/apache/accumulo/core/security/CredentialsTest.java > 4f8079e > > server/base/src/main/java/org/apache/accumulo/server/client/ClientServiceHandler.java > 3571d7f > > server/base/src/main/java/org/apache/accumulo/server/security/SecurityOperation.java > c2a7001 > > Diff: https://reviews.apache.org/r/20702/diff/ > > > Testing > ------- > > unit tests passed. ITs running. > > > Thanks, > > Sean Busbey > >
