[HBASE-26629] Add expiration for long time vacant scanners in Thrift2

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 2.5.0, 3.0.0-alpha-3, 2.4.10
• Component/s: Performance, Thrift
• Hadoop Flags: Reviewed
• Assignee: Yutong Xiao
• Reporter: Yutong Xiao
• Created: 27/Dec/21 08:14
• Updated: 03/Jan/22 19:18
• Resolved: 02/Jan/22 13:32

Description

In thrift1 implementation, the ThriftHBaseSerivceHandler holds a Cache with an expire time after access. This will make the long time vacant scanners be collected by gc. However, in thrift2 we do not have this feature and only use a map to store scanners and we need the client close the scanner manually. If not, the expired scanners will live in memory forever. In this case, I applied the cache expiration to thrift2 service.

Issue Links

• links to: GitHub Pull Request #3984