> On Jan. 21, 2015, 2:23 a.m., Abraham Elmahrek wrote: > > client/src/main/java/org/apache/sqoop/client/request/ResourceRequest.java, > > line 228 > > <https://reviews.apache.org/r/30065/diff/1/?file=826093#file826093line228> > > > > the user.name property is the system name I think. If the user has > > authenticated with Kerberos, it may not be the same as their system name. > > How about UGI.getShortUserName()?
Add comment to clarify this issue. - richard ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30065/#review68847 ----------------------------------------------------------- On Jan. 20, 2015, 8:42 a.m., richard zhou wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/30065/ > ----------------------------------------------------------- > > (Updated Jan. 20, 2015, 8:42 a.m.) > > > Review request for Sqoop. > > > Repository: sqoop-sqoop2 > > > Description > ------- > > SQOOP-2026 > SQOOP2: Make getUserName function in RequestContext support Kerberos > > > Diffs > ----- > > client/src/main/java/org/apache/sqoop/client/request/ResourceRequest.java > aa5fd3598bdbbba95b640bb51b9e01529feade04 > server/src/main/java/org/apache/sqoop/server/RequestContext.java > d0963f5bcd09a5f31601f677304165d513cc7de3 > > Diff: https://reviews.apache.org/r/30065/diff/ > > > Testing > ------- > > local and integration test > > > Thanks, > > richard zhou > >
