[HBASE-4556] Fix all incorrect uses of InternalScanner.next(...)

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 0.92.0, 0.94.0
• Assignee: Lars Hofhansl
• Reporter: Lars Hofhansl
• Created: 08/Oct/11 05:31
• Updated: 20/Nov/15 11:54
• Resolved: 13/Oct/11 18:14

Description

There are cases all over the code where InternalScanner.next(...) is not used correctly.

I see this a lot:

while(scanner.next(...)) {
}

The correct pattern is:

boolean more = false;
do {
   more = scanner.next(...);
} while (more);