hive-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Arnaud Linz (Jira)" <j...@apache.org>
Subject [jira] [Updated] (HIVE-23105) HiveServer2 regression breaks getUpdateCount / getMoreResult API contract
Date Mon, 30 Mar 2020 11:53:00 GMT

     [ https://issues.apache.org/jira/browse/HIVE-23105?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Arnaud Linz updated HIVE-23105:
-------------------------------
    Description: 
Migrating from CDH 5.16 (Hive 1.1.0+) to CDH 6.3 (Hive 2.1.1+) introduced a regression in
the the JDBC driver.

It was detected in a "agnostic" jdbc handling service which works for several DBMS including
Teradata, Impala, and the former Hive driver.

 

java.sql.Statement JDBC Interface method :
{code:java}
 /** 
     *  Retrieves the current result as an update count; 
     *  if the result is a <code>ResultSet</code> object or there are
no more results, -1 
     *  is returned. This method should be called only once per result. 
     * 
     * @return the current result as an update count; -1 if the current result is a 
     * <code>ResultSet</code> object or there are no more results 
     * @exception SQLException if a database access error occurs or 
     * this method is called on a closed <code>Statement</code> 
     * @see #execute 
     */ 
    int getUpdateCount() throws SQLException; {code}
    Does not return -1 when it should, it rather throws :

 
{code:java}
Caused by: java.sql.SQLException: org.apache.thrift.protocol.TProtocolException: Required field 'operationHandle' is unset! Struct:TGetOperationStatusReq(operationHandle:null) 
    at org.apache.hive.jdbc.HiveStatement.waitForOperationToComplete(HiveStatement.java:395) 
    at org.apache.hive.jdbc.HiveStatement.getUpdateCount(HiveStatement.java:688) 
    ... 30 more 
Caused by: org.apache.thrift.protocol.TProtocolException: Required field 'operationHandle' is unset! Struct:TGetOperationStatusReq(operationHandle:null) 
    at org.apache.hive.service.rpc.thrift.TGetOperationStatusReq.validate(TGetOperationStatusReq.java:294) 
    at org.apache.hive.service.rpc.thrift.TCLIService$GetOperationStatus_args.validate(TCLIService.java:12587) 
    at org.apache.hive.service.rpc.thrift.TCLIService$GetOperationStatus_args$GetOperationStatus_argsStandardScheme.write(TCLIService.java:12644) 
    at org.apache.hive.service.rpc.thrift.TCLIService$GetOperationStatus_args$GetOperationStatus_argsStandardScheme.write(TCLIService.java:12613) 
    at org.apache.hive.service.rpc.thrift.TCLIService$GetOperationStatus_args.write(TCLIService.java:12564) 
    at org.apache.thrift.TServiceClient.sendBase(TServiceClient.java:71) 
    at org.apache.thrift.TServiceClient.sendBase(TServiceClient.java:62) 
    at org.apache.hive.service.rpc.thrift.TCLIService$Client.send_GetOperationStatus(TCLIService.java:461) 
    at org.apache.hive.service.rpc.thrift.TCLIService$Client.GetOperationStatus(TCLIService.java:453) 
    at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) 
    at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62) 
    at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) 
    at java.lang.reflect.Method.invoke(Method.java:498) 
    at org.apache.hive.jdbc.HiveConnection$SynchronizedHandler.invoke(HiveConnection.java:1415) 
    at com.sun.proxy.$Proxy20.GetOperationStatus(Unknown Source) 
    at org.apache.hive.jdbc.HiveStatement.waitForOperationToComplete(HiveStatement.java:364) 
    ... 33 more {code}
 

And method:
{code:java}
    /** 
     * Moves to this <code>Statement</code> object's next result, returns 
     * <code>true</code> if it is a <code>ResultSet</code> object,
and 
     * implicitly closes any current <code>ResultSet</code> 
     * object(s) obtained with the method <code>getResultSet</code>. 
     * 
     * <P>There are no more results when the following is true: 
     * <PRE>{@code 
     *     // stmt is a Statement object 
     *     ((stmt.getMoreResults() == false) && (stmt.getUpdateCount() ==
-1)) 
     * }</PRE> 
     * 
     * @return <code>true</code> if the next result is a <code>ResultSet</code> 
     *         object; <code>false</code> if it is an update count or there are 
     *         no more results 
     * @exception SQLException if a database access error occurs or 
     * this method is called on a closed <code>Statement</code> 
     * @see #execute 
     */ 
    boolean getMoreResults() throws SQLException; 
{code}
Always returns true if the statement is not a result set whereas false is expected.

 

 

 

 

  was:
Migrating from CDH 5.16 (Hive 1.1.0+) to CDH 6.3 (Hive 2.1.1+) introduced a regression in
the the JDBC driver.

It was detected in a "agnostic" jdbc handling service which works for several DBMS including
Teradata, Impala, and the former Hive driver.

 

java.sql.Statement JDBC Interface method :
{code:java}
 /** 
     *  Retrieves the current result as an update count; 
     *  if the result is a <code>ResultSet</code> object or there are
no more results, -1 
     *  is returned. This method should be called only once per result. 
     * 
     * @return the current result as an update count; -1 if the current result is a 
     * <code>ResultSet</code> object or there are no more results 
     * @exception SQLException if a database access error occurs or 
     * this method is called on a closed <code>Statement</code> 
     * @see #execute 
     */ 
    int getUpdateCount() throws SQLException; {code}
    Does not return -1 when it should, it rather throws :

 
{code:java}
Caused by: java.sql.SQLException: org.apache.thrift.protocol.TProtocolException: Required field 'operationHandle' is unset! Struct:TGetOperationStatusReq(operationHandle:null) 
    at org.apache.hive.jdbc.HiveStatement.waitForOperationToComplete(HiveStatement.java:395) 
    at org.apache.hive.jdbc.HiveStatement.getUpdateCount(HiveStatement.java:688) 
    ... 30 more 
Caused by: org.apache.thrift.protocol.TProtocolException: Required field 'operationHandle' is unset! Struct:TGetOperationStatusReq(operationHandle:null) 
    at org.apache.hive.service.rpc.thrift.TGetOperationStatusReq.validate(TGetOperationStatusReq.java:294) 
    at org.apache.hive.service.rpc.thrift.TCLIService$GetOperationStatus_args.validate(TCLIService.java:12587) 
    at org.apache.hive.service.rpc.thrift.TCLIService$GetOperationStatus_args$GetOperationStatus_argsStandardScheme.write(TCLIService.java:12644) 
    at org.apache.hive.service.rpc.thrift.TCLIService$GetOperationStatus_args$GetOperationStatus_argsStandardScheme.write(TCLIService.java:12613) 
    at org.apache.hive.service.rpc.thrift.TCLIService$GetOperationStatus_args.write(TCLIService.java:12564) 
    at org.apache.thrift.TServiceClient.sendBase(TServiceClient.java:71) 
    at org.apache.thrift.TServiceClient.sendBase(TServiceClient.java:62) 
    at org.apache.hive.service.rpc.thrift.TCLIService$Client.send_GetOperationStatus(TCLIService.java:461) 
    at org.apache.hive.service.rpc.thrift.TCLIService$Client.GetOperationStatus(TCLIService.java:453) 
    at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) 
    at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62) 
    at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) 
    at java.lang.reflect.Method.invoke(Method.java:498) 
    at org.apache.hive.jdbc.HiveConnection$SynchronizedHandler.invoke(HiveConnection.java:1415) 
    at com.sun.proxy.$Proxy20.GetOperationStatus(Unknown Source) 
    at org.apache.hive.jdbc.HiveStatement.waitForOperationToComplete(HiveStatement.java:364) 
    ... 33 more {code}
 

And method:
{code:java}
    /** 
     * Moves to this <code>Statement</code> object's next result, returns 
     * <code>true</code> if it is a <code>ResultSet</code> object,
and 
     * implicitly closes any current <code>ResultSet</code> 
     * object(s) obtained with the method <code>getResultSet</code>. 
     * 
     * <P>There are no more results when the following is true: 
     * <PRE>{@code 
     *     // stmt is a Statement object 
     *     ((stmt.getMoreResults() == false) && (stmt.getUpdateCount() ==
-1)) 
     * }</PRE> 
     * 
     * @return <code>true</code> if the next result is a <code>ResultSet</code> 
     *         object; <code>false</code> if it is an update count or there are 
     *         no more results 
     * @exception SQLException if a database access error occurs or 
     * this method is called on a closed <code>Statement</code> 
     * @see #execute 
     */ 
    boolean getMoreResults() throws SQLException; 
{code}
Always returns true if the statement is not a result set whereas false is expected (especially
since the javadoc's ((stmt.getMoreResults() == false) && (stmt.getUpdateCount() ==
-1)) throws an Exception...) 

 

 

 

 


> HiveServer2 regression breaks getUpdateCount / getMoreResult API contract
> -------------------------------------------------------------------------
>
>                 Key: HIVE-23105
>                 URL: https://issues.apache.org/jira/browse/HIVE-23105
>             Project: Hive
>          Issue Type: Bug
>          Components: JDBC
>    Affects Versions: 2.1.1
>            Reporter: Arnaud Linz
>            Priority: Major
>
> Migrating from CDH 5.16 (Hive 1.1.0+) to CDH 6.3 (Hive 2.1.1+) introduced a regression
in the the JDBC driver.
> It was detected in a "agnostic" jdbc handling service which works for several DBMS including
Teradata, Impala, and the former Hive driver.
>  
> java.sql.Statement JDBC Interface method :
> {code:java}
>  /** 
>      *  Retrieves the current result as an update count; 
>      *  if the result is a <code>ResultSet</code> object or there are
no more results, -1 
>      *  is returned. This method should be called only once per result. 
>      * 
>      * @return the current result as an update count; -1 if the current result is a 
>      * <code>ResultSet</code> object or there are no more results 
>      * @exception SQLException if a database access error occurs or 
>      * this method is called on a closed <code>Statement</code> 
>      * @see #execute 
>      */ 
>     int getUpdateCount() throws SQLException; {code}
>     Does not return -1 when it should, it rather throws :
>  
> {code:java}
> Caused by: java.sql.SQLException: org.apache.thrift.protocol.TProtocolException: Required field 'operationHandle' is unset! Struct:TGetOperationStatusReq(operationHandle:null) 
>     at org.apache.hive.jdbc.HiveStatement.waitForOperationToComplete(HiveStatement.java:395) 
>     at org.apache.hive.jdbc.HiveStatement.getUpdateCount(HiveStatement.java:688) 
>     ... 30 more 
> Caused by: org.apache.thrift.protocol.TProtocolException: Required field 'operationHandle' is unset! Struct:TGetOperationStatusReq(operationHandle:null) 
>     at org.apache.hive.service.rpc.thrift.TGetOperationStatusReq.validate(TGetOperationStatusReq.java:294) 
>     at org.apache.hive.service.rpc.thrift.TCLIService$GetOperationStatus_args.validate(TCLIService.java:12587) 
>     at org.apache.hive.service.rpc.thrift.TCLIService$GetOperationStatus_args$GetOperationStatus_argsStandardScheme.write(TCLIService.java:12644) 
>     at org.apache.hive.service.rpc.thrift.TCLIService$GetOperationStatus_args$GetOperationStatus_argsStandardScheme.write(TCLIService.java:12613) 
>     at org.apache.hive.service.rpc.thrift.TCLIService$GetOperationStatus_args.write(TCLIService.java:12564) 
>     at org.apache.thrift.TServiceClient.sendBase(TServiceClient.java:71) 
>     at org.apache.thrift.TServiceClient.sendBase(TServiceClient.java:62) 
>     at org.apache.hive.service.rpc.thrift.TCLIService$Client.send_GetOperationStatus(TCLIService.java:461) 
>     at org.apache.hive.service.rpc.thrift.TCLIService$Client.GetOperationStatus(TCLIService.java:453) 
>     at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) 
>     at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62) 
>     at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) 
>     at java.lang.reflect.Method.invoke(Method.java:498) 
>     at org.apache.hive.jdbc.HiveConnection$SynchronizedHandler.invoke(HiveConnection.java:1415) 
>     at com.sun.proxy.$Proxy20.GetOperationStatus(Unknown Source) 
>     at org.apache.hive.jdbc.HiveStatement.waitForOperationToComplete(HiveStatement.java:364) 
>     ... 33 more {code}
>  
> And method:
> {code:java}
>     /** 
>      * Moves to this <code>Statement</code> object's next result, returns 
>      * <code>true</code> if it is a <code>ResultSet</code> object,
and 
>      * implicitly closes any current <code>ResultSet</code> 
>      * object(s) obtained with the method <code>getResultSet</code>. 
>      * 
>      * <P>There are no more results when the following is true: 
>      * <PRE>{@code 
>      *     // stmt is a Statement object 
>      *     ((stmt.getMoreResults() == false) && (stmt.getUpdateCount()
== -1)) 
>      * }</PRE> 
>      * 
>      * @return <code>true</code> if the next result is a <code>ResultSet</code> 
>      *         object; <code>false</code> if it is an update
count or there are 
>      *         no more results 
>      * @exception SQLException if a database access error occurs or 
>      * this method is called on a closed <code>Statement</code> 
>      * @see #execute 
>      */ 
>     boolean getMoreResults() throws SQLException; 
> {code}
> Always returns true if the statement is not a result set whereas false is expected.
>  
>  
>  
>  



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Mime
View raw message