Re: Review Request 15435: Add long polling to asynchronous execution in HiveServer2

2014-02-20 Thread Thejas Nair
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15435/#review35050 --- Ship it! Ship It! - Thejas Nair On Feb. 20, 2014, 2:49 a.m., Vai

Re: Review Request 15435: Add long polling to asynchronous execution in HiveServer2

2014-02-19 Thread Vaibhav Gumashta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15435/ --- (Updated Feb. 20, 2014, 2:49 a.m.) Review request for hive, Carl Steinbach and

Re: Review Request 15435: Add long polling to asynchronous execution in HiveServer2

2014-02-19 Thread Thejas Nair
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15435/#review34933 --- service/src/java/org/apache/hive/service/cli/operation/ExecuteState

Re: Review Request 15435: Add long polling to asynchronous execution in HiveServer2

2014-02-18 Thread Vaibhav Gumashta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15435/ --- (Updated Feb. 18, 2014, 1:16 p.m.) Review request for hive, Carl Steinbach and

Re: Review Request 15435: Add long polling to asynchronous execution in HiveServer2

2013-11-13 Thread Vaibhav Gumashta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15435/ --- (Updated Nov. 14, 2013, 12:02 a.m.) Review request for hive, Carl Steinbach and

Re: Review Request 15435: Add long polling to asynchronous execution in HiveServer2

2013-11-13 Thread Vaibhav Gumashta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15435/ --- (Updated Nov. 13, 2013, 11:58 p.m.) Review request for hive, Carl Steinbach and

Re: Review Request 15435: Add long polling to asynchronous execution in HiveServer2

2013-11-13 Thread Vaibhav Gumashta
> On Nov. 12, 2013, 10:27 p.m., Carl Steinbach wrote: > > service/src/test/org/apache/hive/service/cli/CLIServiceTest.java, line 402 > > > > > > Please use a switch statement here. We can use a switch here but in java

Re: Review Request 15435: Add long polling to asynchronous execution in HiveServer2

2013-11-12 Thread Carl Steinbach
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15435/#review28749 --- service/src/java/org/apache/hive/service/cli/operation/SQLOperation

Re: Review Request 15435: Add long polling to asynchronous execution in HiveServer2

2013-11-11 Thread Vaibhav Gumashta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15435/ --- (Updated Nov. 12, 2013, 2:16 a.m.) Review request for hive, Carl Steinbach and

Re: Review Request 15435: Add long polling to asynchronous execution in HiveServer2

2013-11-11 Thread Vaibhav Gumashta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15435/ --- (Updated Nov. 12, 2013, 12:06 a.m.) Review request for hive, Carl Steinbach and

Review Request 15435: Add long polling to asynchronous execution in HiveServer2

2013-11-11 Thread Vaibhav Gumashta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15435/ --- Review request for hive, Carl Steinbach and Thejas Nair. Bugs: HIVE-5217 ht