> On Sept. 8, 2015, 3:57 p.m., Jason Altekruse wrote: > > exec/java-exec/src/main/java/org/apache/drill/exec/ExecConstants.java, line > > 265 > > <https://reviews.apache.org/r/37910/diff/1/?file=1058412#file1058412line265> > > > > These should both be updated to be subclasses of AdminOptionValidator, > > so that we can veryify that the string passed is the correct format when it > > is set. > > Venki Korukanti wrote: > Currently the format is very loose, basically a comma separated list of > users or groups. Not sure what we want to validate here. Enforcing a certain > format on the user name is risky given the format of userName/groupName is > depended on the underlying system.
Sounds reasonable enough to me +1 - Jason ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37910/#review97760 ----------------------------------------------------------- On Aug. 28, 2015, 11 p.m., Venki Korukanti wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/37910/ > ----------------------------------------------------------- > > (Updated Aug. 28, 2015, 11 p.m.) > > > Review request for drill and Jason Altekruse. > > > Repository: drill-git > > > Description > ------- > > Please see DRILL-3622 for details. > > > Diffs > ----- > > exec/java-exec/src/main/java/org/apache/drill/exec/ExecConstants.java > 140e9a8 > exec/java-exec/src/main/java/org/apache/drill/exec/ops/QueryContext.java > c3cd4e6 > > exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/handlers/SetOptionHandler.java > 19c9f7f > > exec/java-exec/src/main/java/org/apache/drill/exec/server/options/SystemOptionManager.java > 2d41740 > > exec/java-exec/src/main/java/org/apache/drill/exec/server/options/TypeValidators.java > b8597b7 > > exec/java-exec/src/main/java/org/apache/drill/exec/util/ImpersonationUtil.java > aa766be > exec/java-exec/src/test/java/org/apache/drill/BaseTestQuery.java d428920 > > exec/java-exec/src/test/java/org/apache/drill/exec/rpc/user/security/UserAuthenticatorTestImpl.java > c89471f > > exec/java-exec/src/test/java/org/apache/drill/exec/server/TestOptionsAuthEnabled.java > PRE-CREATION > > exec/java-exec/src/test/java/org/apache/drill/exec/server/rest/RootResource.java > 59adad9 > > Diff: https://reviews.apache.org/r/37910/diff/ > > > Testing > ------- > > Added unittest. Regression suites currently running. > > > Thanks, > > Venki Korukanti > >
