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

Joep Rottinghuis commented on YARN-5070:
----------------------------------------

Perhaps instead of

{code}
int limit = getBatchLimit(scannerContext);
{code}
in #nextInternal(List<Cell> cells, ScannerContext scannerContext)

we should use
{code}
int limit = regionScanner.getBatch();
{code}
See: 
https://hbase.apache.org/devapidocs/org/apache/hadoop/hbase/regionserver/RegionScanner.html#getBatch%28%29

then we can probably do away with the reflection.
Btw. should we also honor regionScanner.getMaxResultSize()?

> upgrade HBase version for first merge
> -------------------------------------
>
>                 Key: YARN-5070
>                 URL: https://issues.apache.org/jira/browse/YARN-5070
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: timelineserver
>    Affects Versions: YARN-2928
>            Reporter: Sangjin Lee
>            Assignee: Vrushali C
>            Priority: Critical
>              Labels: yarn-2928-1st-milestone
>         Attachments: YARN-5070-YARN-2928.01.patch, 
> YARN-5070-YARN-2928.02.patch
>
>
> Currently we set the HBase version for the timeline service storage to 1.0.1. 
> This is a fairly old version, and there are reasons to upgrade to a newer 
> version. We should upgrade it.



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

---------------------------------------------------------------------
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org

Reply via email to