[PHOENIX-6872] Use ServerUtil.ConnectionFactory.getConnection() in UngroupedAggregateRegionScanner

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 5.2.0, 5.1.4
• Component/s: core
• Assignee: Istvan Toth
• Reporter: Istvan Toth
• Created: 09/Feb/23 13:12
• Updated: 11/Feb/23 09:02
• Resolved: 10/Feb/23 18:51

Description

We've optimized Connection handling by using a PhoenixConnection from the CQSI.

However, we only need an (H)Connection here, so we could use org.apache.phoenix.util.ServerUtil.ConnectionFactory.getConnection(ConnectionType, RegionCoprocessorEnvironment)

and avoid instantiating a PhoenixConnection.

We already avoid the very expensive Connection creation, so this is just a small incremetal improvement over the current code.

Issue Links

• is related to: PHOENIX-6834 Use Pooled HConnection for Server Side Upsert Select
• relates to: PHOENIX-6838 Review unpooled HConnection usage
• links to: GitHub Pull Request #1562