-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15151/
-----------------------------------------------------------
(Updated Nov. 15, 2013, 7:59 p.m.)
Review request for hive, Carl Steinbach, Prasad Mujumdar, and Thejas Nair.
Changes
-------
Latest feedback.
Bugs: HIVE-5230
https://issues.apache.org/jira/browse/HIVE-5230
Repository: hive-git
Description
-------
[HIVE-4617|https://issues.apache.org/jira/browse/HIVE-4617] provides support
for async execution in HS2. When a background thread gets an error, currently
the client can only poll for the operation state and also the error with its
stacktrace is logged. However, it will be useful to provide a richer error
response like thrift API does with TStatus (which is constructed while building
a Thrift response object).
Diffs (updated)
-----
jdbc/src/java/org/apache/hive/jdbc/HiveConnection.java ef39573
jdbc/src/java/org/apache/hive/jdbc/HiveStatement.java fce19bf
service/if/TCLIService.thrift 1f49445
service/src/java/org/apache/hive/service/cli/CLIService.java 8c85386
service/src/java/org/apache/hive/service/cli/CLIServiceClient.java 14ef54f
service/src/java/org/apache/hive/service/cli/EmbeddedCLIServiceClient.java
9dca874
service/src/java/org/apache/hive/service/cli/HiveSQLException.java 74e8b94
service/src/java/org/apache/hive/service/cli/ICLIService.java f647ce6
service/src/java/org/apache/hive/service/cli/OperationState.java 1ec6bd1
service/src/java/org/apache/hive/service/cli/OperationStatus.java
PRE-CREATION
service/src/java/org/apache/hive/service/cli/operation/Operation.java 6f4b8dc
service/src/java/org/apache/hive/service/cli/operation/OperationManager.java
bcdb67f
service/src/java/org/apache/hive/service/cli/operation/SQLOperation.java
4ee1b74
service/src/java/org/apache/hive/service/cli/session/SessionManager.java
f392d62
service/src/java/org/apache/hive/service/cli/thrift/ThriftCLIService.java
9df110e
service/src/java/org/apache/hive/service/cli/thrift/ThriftCLIServiceClient.java
9bb2a0f
service/src/test/org/apache/hive/service/cli/CLIServiceTest.java cd9d99a
service/src/test/org/apache/hive/service/cli/thrift/ThriftCLIServiceTest.java
ff7166d
Diff: https://reviews.apache.org/r/15151/diff/
Testing
-------
Thanks,
Vaibhav Gumashta
