You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@hive.apache.org by Prasad Mujumdar <pr...@cloudera.com> on 2013/08/01 05:23:59 UTC

Re: Review Request 10698: HIVE-4395: Support TFetchOrientation.FIRST for HiveServer2 FetchResults

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/10698/
-----------------------------------------------------------

(Updated Aug. 1, 2013, 3:23 a.m.)


Review request for hive and Carl Steinbach.


Changes
-------

rebased patch, added more test cases for set and dfs commands


Bugs: HIVE-4395
    https://issues.apache.org/jira/browse/HIVE-4395


Repository: hive-git


Description
-------

Support fetch-from-start for hiveserver2 fetch operations. 
 - Handle new fetch orientation for various HS2 operations.
 - Added support to reset the read position in Hive driver
 - Enabled scroll cursors with support for positioning cursor to start of resultset


Diffs (updated)
-----

  jdbc/src/java/org/apache/hive/jdbc/HiveConnection.java 00f4351 
  jdbc/src/java/org/apache/hive/jdbc/HiveQueryResultSet.java 61985d1 
  jdbc/src/java/org/apache/hive/jdbc/HiveStatement.java 982ceb8 
  jdbc/src/test/org/apache/hive/jdbc/TestJdbcDriver2.java 1042125 
  ql/src/java/org/apache/hadoop/hive/ql/Context.java 2a3ee24 
  ql/src/java/org/apache/hadoop/hive/ql/Driver.java 2a6b944 
  ql/src/java/org/apache/hadoop/hive/ql/exec/FetchTask.java df2ccf1 
  ql/src/java/org/apache/hadoop/hive/ql/processors/DfsProcessor.java ce54e0c 
  service/src/java/org/apache/hive/service/cli/operation/DfsOperation.java a8b8ed4 
  service/src/java/org/apache/hive/service/cli/operation/GetCatalogsOperation.java 581e69c 
  service/src/java/org/apache/hive/service/cli/operation/GetColumnsOperation.java af87a90 
  service/src/java/org/apache/hive/service/cli/operation/GetFunctionsOperation.java 0fe01c0 
  service/src/java/org/apache/hive/service/cli/operation/GetSchemasOperation.java bafe40c 
  service/src/java/org/apache/hive/service/cli/operation/GetTableTypesOperation.java eaf867e 
  service/src/java/org/apache/hive/service/cli/operation/GetTablesOperation.java d9d0e9c 
  service/src/java/org/apache/hive/service/cli/operation/GetTypeInfoOperation.java 2daa9cd 
  service/src/java/org/apache/hive/service/cli/operation/HiveCommandOperation.java 0a8825e 
  service/src/java/org/apache/hive/service/cli/operation/Operation.java 6f4b8dc 
  service/src/java/org/apache/hive/service/cli/operation/SQLOperation.java 976a1ef 

Diff: https://reviews.apache.org/r/10698/diff/


Testing
-------

Added new JDBC test cases.


Thanks,

Prasad Mujumdar