----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19801/#review38968 -----------------------------------------------------------
service/src/java/org/apache/hive/service/cli/thrift/ThriftHttpServlet.java <https://reviews.apache.org/r/19801/#comment71348> Can we use proxyUserNameParam for doAsQueryParam? doAs in hive world is different than proxy user like we discussed offline. service/src/java/org/apache/hive/service/cli/thrift/ThriftHttpServlet.java <https://reviews.apache.org/r/19801/#comment71349> Similar to the previous comment: I think it'll be good to change the method name to getProxyUserNameParam from getDoAsQueryParam. - Vaibhav Gumashta On March 28, 2014, 9:08 p.m., dilli dorai wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/19801/ > ----------------------------------------------------------- > > (Updated March 28, 2014, 9:08 p.m.) > > > Review request for hive, Thejas Nair and Vaibhav Gumashta. > > > Bugs: HIVE-6738 > https://issues.apache.org/jira/browse/HIVE-6738 > > > Repository: hive-git > > > Description > ------- > > see the jira HIVE-6738 > > > Diffs > ----- > > service/src/java/org/apache/hive/service/cli/session/SessionManager.java > 7f6687e > service/src/java/org/apache/hive/service/cli/thrift/ThriftCLIService.java > 58f3e3b > service/src/java/org/apache/hive/service/cli/thrift/ThriftHttpServlet.java > c579db5 > > Diff: https://reviews.apache.org/r/19801/diff/ > > > Testing > ------- > > see the attachment to Jira HIVE-6738 > https://issues.apache.org/jira/secure/attachment/12637059/hive-6738-req-impl-verify.md > > > Thanks, > > dilli dorai > >
