> On Aug. 27, 2015, 8:57 p.m., Chao Sun wrote: > > Patch looks good. Just one question: instead of populating the user name in > > several places, is it possible to use the one stored in SessionState (by > > calling SessionState.get().getUserName() before creating the Driver)? > > Aihua Xu wrote: > Didn't know there is one in SessionState. Let me take a look how those > usernames are co-related. > > Aihua Xu wrote: > Updated to use the userName saved in the sessionState and it seems to be > the better way to save and use userName. While I do get lost to track if the > userName in the session would be set properly for all the pathes.
I think the SessionState is setup before the query is executed and should be available during the whole lifetime of it. - Chao ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37810/#review96749 ----------------------------------------------------------- On Aug. 28, 2015, 3:16 p.m., Aihua Xu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/37810/ > ----------------------------------------------------------- > > (Updated Aug. 28, 2015, 3:16 p.m.) > > > Review request for hive and Chao Sun. > > > Repository: hive-git > > > Description > ------- > > HIVE-10021 "Alter index rebuild" statements submitted through HiveServer2 > fail when Sentry is enabled > > > Diffs > ----- > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/IndexUtils.java 92cae67 > > Diff: https://reviews.apache.org/r/37810/diff/ > > > Testing > ------- > > > Thanks, > > Aihua Xu > >
