----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34883/#review86364 -----------------------------------------------------------
Ship it! LGTM. shell/src/main/java/org/apache/sqoop/shell/RevokePrivilegeFunction.java <https://reviews.apache.org/r/34883/#comment138330> --resource instead of Resource shell/src/main/java/org/apache/sqoop/shell/RevokePrivilegeFunction.java <https://reviews.apache.org/r/34883/#comment138329> --resource-type instead of Resource type shell/src/main/java/org/apache/sqoop/shell/RevokePrivilegeFunction.java <https://reviews.apache.org/r/34883/#comment138328> --action instead of Action - Abraham Elmahrek On June 2, 2015, 1:55 a.m., Dian Fu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/34883/ > ----------------------------------------------------------- > > (Updated June 2, 2015, 1:55 a.m.) > > > Review request for Sqoop. > > > Repository: sqoop-sqoop2 > > > Description > ------- > > SQOOP2: Revoke all privilege > > > Diffs > ----- > > client/src/main/java/org/apache/sqoop/client/SqoopClient.java d5c4a8a > shell/src/main/java/org/apache/sqoop/shell/RevokePrivilegeFunction.java > 802d206 > > Diff: https://reviews.apache.org/r/34883/diff/ > > > Testing > ------- > > > Thanks, > > Dian Fu > >
