Re: Review Request 11326: HIVE-4588: Support session level hooks for HiveServer2

2013-08-22 Thread Prasad Mujumdar
> On Aug. 22, 2013, 8:38 a.m., Navis Ryu wrote: > > ql/src/java/org/apache/hadoop/hive/ql/Driver.java, line 429 > > > > > > This is not related to this issue, but would it be better to use > > interface (HiveSemanticA

Re: Review Request 11326: HIVE-4588: Support session level hooks for HiveServer2

2013-08-22 Thread Prasad Mujumdar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11326/ --- (Updated Aug. 22, 2013, 6:54 p.m.) Review request for hive and Navis Ryu. Cha

Re: Review Request 11326: HIVE-4588: Support session level hooks for HiveServer2

2013-08-22 Thread Prasad Mujumdar
> On Aug. 22, 2013, 8:38 a.m., Navis Ryu wrote: > > ql/src/java/org/apache/hadoop/hive/ql/Driver.java, line 1112 > > > > > > Isn't this message should be logged? ah, yes. sorry I missed that. will update the patch.

Re: Review Request 11326: HIVE-4588: Support session level hooks for HiveServer2

2013-08-22 Thread Navis Ryu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11326/#review25410 --- ql/src/java/org/apache/hadoop/hive/ql/Driver.java

Re: Review Request 11326: HIVE-4588: Support session level hooks for HiveServer2

2013-08-22 Thread Prasad Mujumdar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11326/ --- (Updated Aug. 22, 2013, 8:02 a.m.) Review request for hive. Changes ---

Re: Review Request 11326: HIVE-4588: Support session level hooks for HiveServer2

2013-08-22 Thread Prasad Mujumdar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11326/ --- (Updated Aug. 22, 2013, 8:03 a.m.) Review request for hive and Navis Ryu. Cha

Re: Review Request 11326: HIVE-4588: Support session level hooks for HiveServer2

2013-08-01 Thread Prasad Mujumdar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11326/ --- (Updated Aug. 1, 2013, 7:12 a.m.) Review request for hive. Changes --- R

Re: Review Request 11326: HIVE-4588: Support session level hooks for HiveServer2

2013-06-26 Thread Prasad Mujumdar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11326/ --- (Updated June 27, 2013, 12:39 a.m.) Review request for hive. Changes ---