[HBASE-13193] RegionScannerImpl filters should not be reset if a partial Result is returned

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.1.0, 2.0.0
• Hadoop Flags: Reviewed
• Assignee: Jonathan Lawlor
• Reporter: Jonathan Lawlor
• Created: 10/Mar/15 18:12
• Updated: 03/Jun/15 15:28
• Resolved: 16/Mar/15 20:27

Description

In RegionScannerImpl the filters get reset during calls to nextRaw:

public NextState nextRaw(List<Cell> outResults, int batchLimit, long remainingResultSize)
        throws IOException {
  ...
  resetFilters();

  if (isFilterDoneInternal()) {
  ...
}

This creates a problem when returning partial Results because filters should only be reset in between rows (i.e. after all of the cells for a particular row have been returned). If a partial Result is returned, there are still cells in the row that have not been evaluated yet and we should avoid resetting the filter since it will wipe away all state information for that filter.

Issue Links

• is part of: HBASE-11544 [Ergonomics] hbase.client.scanner.caching is dogged and will try to return batch even if it means OOME