[HBASE-25723] Temporarily remove the trace support for RegionScanner.next

• Type: Sub-task
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 3.0.0-alpha-1
• Component/s: scan, tracing
• Hadoop Flags: Reviewed
• Assignee: Duo Zhang
• Reporter: Duo Zhang
• Created: 01/Apr/21 16:06
• Updated: 02/Sep/21 18:51
• Resolved: 03/Apr/21 15:08

Description

Please see the test result in HBASE-25658, tracing RegionScanner.next could lead to a 6% performance downgrading even if we do not enable tracing.

I think the problem here is that, a single scan rpc call could lead to a bunch of RegionScanner.next. So if trace is enabled, there will be much more performance impact as we need to store a bunch of spans(maybe this could be limited by some parameters?).

Anyway, let's remove it for now to land the basic trace framework first. Can improve later.

Issue Links

• links to: GitHub Pull Request #3119