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

Naveen Gangam updated HIVE-22853:
---------------------------------
    Status: Patch Available  (was: In Progress)

HiveConnection.OpenSession() sets the fetchSize=1000 into the map passed to 
server side on OpenSession. In response, the server checks to see if the 
sessionConf has a fetchSize set, if set, sends it back in the response. 
Essentially, sending back the value sent by the client. It now sends back the 
server side conf value.

The HiveConnection() has a fetchSize thats fetched from server-side conf now 
which is private thats used when a statement is created. However, this is set 
during OpenSession where the server passes back the value in a response. So 
there is no good way to refresh when beeline resets it via SET command as it 
does not have a public mutator. Instead, I am using a beeline local variable to 
track the local value. 
Ideally, this should be per connection/session config but we cannot violate the 
JDBC spec for HiveConnection.

Not as clean as I would like it to be. 

> Beeline should use HS2 server defaults for fetchSize
> ----------------------------------------------------
>
>                 Key: HIVE-22853
>                 URL: https://issues.apache.org/jira/browse/HIVE-22853
>             Project: Hive
>          Issue Type: Bug
>          Components: Beeline
>    Affects Versions: 4.0.0
>            Reporter: Naveen Gangam
>            Assignee: Naveen Gangam
>            Priority: Major
>         Attachments: HIVE-22853.patch
>
>
> Currently beeline uses a hard coded default of 1000 rows for fetchSize. This 
> default value is different from what the server has set. While the beeline 
> user can reset the value via set command, its cumbersome to change the 
> workloads.
> Rather it should default to the server-side value and set should be used to 
> override within the session.



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

Reply via email to