[jira] [Resolved] (PHOENIX-7387) SnapshotScanner's next method is ignoring the boolean value from hbase's nextRaw method

["Istvan Toth (Jira)" <ji...@apache.org>]

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

Istvan Toth resolved PHOENIX-7387.
----------------------------------
    Resolution: Fixed

Committed to all active braches.
Thanks for the patch, [~abhradeep.kundu].

> SnapshotScanner's next method is ignoring the boolean value from hbase's nextRaw method
> ---------------------------------------------------------------------------------------
>
>                 Key: PHOENIX-7387
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-7387
>             Project: Phoenix
>          Issue Type: Bug
>          Components: core
>            Reporter: Abhradeep Kundu
>            Assignee: Abhradeep Kundu
>            Priority: Major
>             Fix For: 5.2.1, 5.3.0, 5.1.4
>
>
> The below code (SnapshotScanner.java) is ignoring the boolean value returning from the nextRaw method
> {code:java}
> public Result next() throws IOException {
> values.clear();
> scanner.nextRaw(values);
> statisticsCollector.collectStatistics(values);
> if (values.isEmpty()) {
> //we are done
> return null;
> }
> return Result.create(values);
> }{code}
> Below is the comment in nextRaw method of Hbase
> {code:java}
> /**
>  * Grab the next row's worth of values. This is a special internal method to be called from
>  * coprocessor hooks to avoid expensive setup. Caller must set the thread's readpoint, start and
>  * close a region operation, an synchronize on the scanner object. Caller should maintain and
>  * update metrics. See {@link #nextRaw(List, ScannerContext)}
>  * @param result return output array
>  * @return true if more rows exist after this one, false if scanner is done
>  * @throws IOException e
>  */
> boolean nextRaw(List<Cell> result) throws IOException; {code}
>  



--
This message was sent by Atlassian Jira
(v8.20.10#820010)