[PHOENIX-6658] Replace HRegion.get() calls

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 5.2.0, 5.1.3
• Assignee: Istvan Toth
• Reporter: Istvan Toth
• Created: 28/Feb/22 16:43
• Updated: 02/Oct/23 05:09
• Resolved: 21/Mar/22 06:59

Description

HBASE-26036 made a change where Region.get() always clones Off-Heap cells before returning them.

In HBase all non-test occurances of this code were changed to create their own RegionScanner to avoid copying the off-heap cells. We should do the same.

This would also let Phoenix run correctly with HBase 2.4.5-2.4.10 , as the fix in HBASE-26036  introduced another bug, HBASE-26777, which causes failures in Phoenix.

Without this change, Phoenix needs at least HBase version 2.4.11, that has the fix for HBASE-26777.

Issue Links

• is related to: HBASE-26036 DBB released too early and dirty data for some operations
• is related to: HBASE-26777 BufferedDataBlockEncoder$OffheapDecodedExtendedCell.deepClone throws UnsupportedOperationException
• is related to: PHOENIX-7057 Potential bug in MetadataEndpointImpl#updateIndexState.
• relates to: PHOENIX-6670 Optimize PhoenixKeyValueUtil#maybeCopyCell
• links to: GitHub Pull Request #1398