[HBASE-11766] Backdoor CoprocessorHConnection is no longer being used for local writes

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 0.98.4
• Fix Version/s: 0.99.0, 0.98.6
• Labels: Phoenix
• Hadoop Flags: Reviewed
• Assignee: Andrew Kyle Purtell
• Reporter: James R. Taylor
• Created: 16/Aug/14 03:22
• Updated: 06/Apr/18 17:51
• Resolved: 25/Aug/14 23:09

Description

There's a backdoor CoprocessorHConnection used to ensure that a batched mutation does not go over the wire and back, but executes immediately locally. This is leveraged by Phoenix during secondary index maintenance (for an ~20% perf improvement). It looks to me like it's no longer used, as the following function is never invoked:

public org.apache.hadoop.hbase.protobuf.generated.ClientProtos.ClientService.BlockingInterface
getClient(ServerName serverName) throws IOException {

It'd be good if feasible to add an HBase unit test to prevent further regressions. For more info, see PHOENIX-1166.

Issue Links

• is related to: HBASE-11819 Unit test for CoprocessorHConnection