[ 
https://issues.apache.org/jira/browse/PHOENIX-1407?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14202233#comment-14202233
 ] 

James Taylor commented on PHOENIX-1407:
---------------------------------------

+1. Looks great, [~gabriel.reid]. Thanks for the quick turnaround. Doing it in 
the constructor is fine.

The scanner caching is set in QueryServicesOptions to 1000 (i.e. 
QueryServicesOptions.DEFAULT_SCAN_CACHE_SIZE), but it would not have been 
overridden by the Statement fetch size.

Please commit to 3.0, 3.2, 4.0, 4.2, and master.

> PhoenixStatement.getFetchSize() appears to never be used
> --------------------------------------------------------
>
>                 Key: PHOENIX-1407
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-1407
>             Project: Phoenix
>          Issue Type: Bug
>            Reporter: James Taylor
>            Assignee: Gabriel Reid
>         Attachments: PHOENIX-1407.patch
>
>
> We should use PhoenixStatement.getFetchSize() or potentially 
> PhoenixResultSet.getFetchSize() to determine what to use for 
> Scan.setCaching(). I'm not seeing this done currently, but maybe I'm just 
> missing it.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to