> On Feb. 23, 2014, 12:36 a.m., Thejas Nair wrote: > > jdbc/src/java/org/apache/hive/jdbc/HiveStatement.java, line 271 > > <https://reviews.apache.org/r/14950/diff/3/?file=498672#file498672line271> > > > > I think we should retry on exceptions that indicate an network > > connection error. TProtocolException seems to be the exception that is > > thrown in such cases.
I think we can address this in a follow up patch. Connection error in case of the current synchronous execution also would result in failure. - Thejas ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14950/#review35234 ----------------------------------------------------------- On Feb. 20, 2014, 9:18 a.m., Vaibhav Gumashta wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/14950/ > ----------------------------------------------------------- > > (Updated Feb. 20, 2014, 9:18 a.m.) > > > Review request for hive and Thejas Nair. > > > Repository: hive-git > > > Description > ------- > > Should be applied on top of: > HIVE-5217 [Add long polling to asynchronous execution in HiveServer2] > HIVE-5229 [Better thread management for HiveServer2 async threads] > HIVE-5230 [Better error reporting by async threads in HiveServer2] > HIVE-5441 [Async query execution doesn't return resultset status] > > > Diffs > ----- > > jdbc/src/java/org/apache/hive/jdbc/HiveStatement.java f0d0c77 > > Diff: https://reviews.apache.org/r/14950/diff/ > > > Testing > ------- > > TestJdbcDriver2 > > > Thanks, > > Vaibhav Gumashta > >
