----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31789/#review75672 -----------------------------------------------------------
Ship it! Ship It! - Abraham Elmahrek On March 6, 2015, 3:05 a.m., richard zhou wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/31789/ > ----------------------------------------------------------- > > (Updated March 6, 2015, 3:05 a.m.) > > > Review request for Sqoop. > > > Repository: sqoop-sqoop2 > > > Description > ------- > > Add ServerName for authorization handler, which will be used by 3rd > implementation, etc. Sentry. > > > Diffs > ----- > > core/src/main/java/org/apache/sqoop/security/AuthorizationHandler.java > 7d60a014fe63459bf3fa923d2302f37192357f43 > core/src/main/java/org/apache/sqoop/security/AuthorizationManager.java > 7f0fe2721def0ea794db2de38538d3a820db9c01 > core/src/main/java/org/apache/sqoop/security/SecurityConstants.java > fde6b60432f270195262def26a6bef8ae9df601b > dist/src/main/server/conf/sqoop.properties > f116e17d2d284b3709744a06af607fdbf9a2ae2b > > security/src/main/java/org/apache/sqoop/security/Authorization/DefaultAuthorizationHandler.java > c60b8a5a38ee065ee1be602b04dc8acf856bf147 > > Diff: https://reviews.apache.org/r/31789/diff/ > > > Testing > ------- > > manual test > > > Thanks, > > richard zhou > >
