hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Vaibhav Gumashta" <vgumas...@hortonworks.com>
Subject Re: Review Request 15151: Better error reporting by async threads in HiveServer2
Date Fri, 15 Nov 2013 20:00:03 GMT

-----------------------------------------------------------
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.


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
-----

  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


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message