[PHOENIX-5839] CompatUtil#setStopRow semantics problem

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 4.16.0
• Component/s: core
• Assignee: Istvan Toth
• Reporter: Istvan Toth
• Created: 14/Apr/20 05:48
• Updated: 25/Feb/21 02:43
• Resolved: 02/Oct/20 14:20

Description

As noticed by gjacoby , the semantics of the CompatUtil#setStopRow method are incorrect for HBase 1.3. Specifically, we invert the semantics of the inclusive flag.

Due to the quirks of the HBase 1.3 scan semantics, the resulting behaviour in the specific cases where it used is correct (in fact this behaviour is needed for correct operation), but when used outside this specific use case (single row scan), it would cause problems.

Find some other solution that solves the backwards compatibility problem, and does not invert the semantics of the call in the range (not single row) scan case.

Issue Links

• relates to: PHOENIX-5799 Inline Index Verification Output API
• links to: GitHub Pull Request #763