----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41914/#review112914 -----------------------------------------------------------
Ship it! - Abraham Fine On Jan. 5, 2016, 2:46 a.m., Jarek Cecho wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/41914/ > ----------------------------------------------------------- > > (Updated Jan. 5, 2016, 2:46 a.m.) > > > Review request for Sqoop. > > > Bugs: SQOOP-2774 > https://issues.apache.org/jira/browse/SQOOP-2774 > > > Repository: sqoop-sqoop2 > > > Description > ------- > > Removed the class RolesBean. > > > Diffs > ----- > > > client/src/main/java/org/apache/sqoop/client/request/AuthorizationResourceRequest.java > ca05d1b > > client/src/main/java/org/apache/sqoop/client/request/SqoopResourceRequests.java > 3109718 > common/src/main/java/org/apache/sqoop/json/RoleBean.java ffafddb > common/src/main/java/org/apache/sqoop/json/RolesBean.java c09ba95 > common/src/test/java/org/apache/sqoop/json/TestRoleBean.java PRE-CREATION > > server/src/main/java/org/apache/sqoop/handler/AuthorizationRequestHandler.java > 1ed63e4 > > Diff: https://reviews.apache.org/r/41914/diff/ > > > Testing > ------- > > Added new unit test, integration tests will be executed as part of precommit > hook. > > > Thanks, > > Jarek Cecho > >
