[HBASE-25644] Scan#setSmall blindly sets ReadType as PREAD

• Type: Bug
• Status: Resolved
• Priority: Critical
• Resolution: Fixed
• Fix Version/s: 3.0.0-alpha-1, 2.2.7, 2.5.0, 2.3.5, 2.4.2
• Labels: phoenix
• Hadoop Flags: Reviewed
• Assignee: Anoop Sam John
• Reporter: Anoop Sam John
• Created: 08/Mar/21 04:15
• Updated: 12/Mar/21 07:20
• Resolved: 08/Mar/21 08:36

Description

setSmall(boolean small) takes a boolean param and it might get called with false also. But with out considering that, we set read type as PREAD.
Phoenix clones Scan object and do
newScan.setSmall(scan.isSmall());
So this makes ALL types of scans from phoenix as PREAD type now. (Even full table scan)

Issue Links

• relates to: PHOENIX-6385 Not to use Scan#setSmall for HBase 2.x versions
• links to: GitHub Pull Request #3032