----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46755/#review130884 -----------------------------------------------------------
jdbc/src/java/org/apache/hive/jdbc/HiveStatement.java (line 345) <https://reviews.apache.org/r/46755/#comment194811> Typo (the extra "+") ql/src/java/org/apache/hadoop/hive/ql/Driver.java (line 1731) <https://reviews.apache.org/r/46755/#comment194807> I should probably have a lock around this, or do null check within plan.setDone. I think the latter might be more optimized as the Driver overall is not thread safe and introducing ad-hoc locks might be a short-sighted approach. ql/src/java/org/apache/hadoop/hive/ql/Driver.java (line 1737) <https://reviews.apache.org/r/46755/#comment194808> Should use logger here. service-rpc/if/TCLIService.thrift (line 461) <https://reviews.apache.org/r/46755/#comment194809> Whitespace service-rpc/if/TCLIService.thrift (line 703) <https://reviews.apache.org/r/46755/#comment194810> Whitespace. service/src/java/org/apache/hive/service/cli/operation/SQLOperation.java (line 102) <https://reviews.apache.org/r/46755/#comment194812> Typo in variable name; change to timeoutExecutor - Vaibhav Gumashta On April 27, 2016, 11:14 p.m., Vaibhav Gumashta wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/46755/ > ----------------------------------------------------------- > > (Updated April 27, 2016, 11:14 p.m.) > > > Review request for hive and Thejas Nair. > > > Bugs: HIVE-4924 > https://issues.apache.org/jira/browse/HIVE-4924 > > > Repository: hive-git > > > Description > ------- > > https://issues.apache.org/jira/browse/HIVE-4924 > > > Diffs > ----- > > itests/hive-unit/src/test/java/org/apache/hive/jdbc/TestJdbcDriver2.java > 965627f > > itests/hive-unit/src/test/java/org/apache/hive/service/cli/session/TestHiveSessionImpl.java > 4d763d2 > jdbc/src/java/org/apache/hive/jdbc/HiveStatement.java 3cc6b74 > ql/src/java/org/apache/hadoop/hive/ql/Driver.java dad43fb > ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistoryImpl.java 0234fd9 > service-rpc/if/TCLIService.thrift 92bcf77 > service-rpc/src/gen/thrift/gen-cpp/TCLIService_types.h 9f937ca > service-rpc/src/gen/thrift/gen-cpp/TCLIService_types.cpp 66f5e8c > > service-rpc/src/gen/thrift/gen-javabean/org/apache/hive/service/rpc/thrift/TExecuteStatementReq.java > 2eb4d09 > > service-rpc/src/gen/thrift/gen-javabean/org/apache/hive/service/rpc/thrift/TOperationState.java > 3fa49b0 > service-rpc/src/gen/thrift/gen-php/Types.php 7f1f99f > service-rpc/src/gen/thrift/gen-py/TCLIService/ttypes.py 3bb20b8 > service-rpc/src/gen/thrift/gen-rb/t_c_l_i_service_types.rb 7208bae > service/src/java/org/apache/hive/service/cli/CLIService.java 4a83e38 > service/src/java/org/apache/hive/service/cli/EmbeddedCLIServiceClient.java > 79e0024 > service/src/java/org/apache/hive/service/cli/ICLIService.java e4aef96 > service/src/java/org/apache/hive/service/cli/OperationState.java 6a67a1d > > service/src/java/org/apache/hive/service/cli/operation/ExecuteStatementOperation.java > b3d9b52 > > service/src/java/org/apache/hive/service/cli/operation/HiveCommandOperation.java > f18dc67 > > service/src/java/org/apache/hive/service/cli/operation/MetadataOperation.java > 77228fa > service/src/java/org/apache/hive/service/cli/operation/Operation.java > 63b1a48 > > service/src/java/org/apache/hive/service/cli/operation/OperationManager.java > 52e4b4d > service/src/java/org/apache/hive/service/cli/operation/SQLOperation.java > 01dd48c > service/src/java/org/apache/hive/service/cli/session/HiveSession.java > 9ea643b > service/src/java/org/apache/hive/service/cli/session/HiveSessionImpl.java > 0cfec7a > > service/src/java/org/apache/hive/service/cli/thrift/RetryingThriftCLIServiceClient.java > b2e0e9e > service/src/java/org/apache/hive/service/cli/thrift/ThriftCLIService.java > 6ede1d7 > > service/src/java/org/apache/hive/service/cli/thrift/ThriftCLIServiceClient.java > 098aea6 > > service/src/test/org/apache/hive/service/cli/thrift/ThriftCLIServiceTest.java > 1740079 > > service/src/test/org/apache/hive/service/cli/thrift/ThriftCliServiceTestWithCookie.java > a1ef1fc > > Diff: https://reviews.apache.org/r/46755/diff/ > > > Testing > ------- > > > Thanks, > > Vaibhav Gumashta > >
