[HBASE-22936] Close memStoreScanners in StoreScanner#updateReaders else memory leak

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Implemented
• Assignee: Unassigned
• Reporter: Michael Stack
• Created: 28/Aug/19 03:13
• Updated: 07/Apr/20 23:27
• Resolved: 02/Oct/19 11:47

Description

Via aoxiang from over on HBASE-22723

+      if (!closeLock.tryLock()) {
+        // no lock acquired.
+        LOG.debug("StoreScanner already has the close lokc. There is no need to updateReaders");
+        return;
+      }
+      // lock acquired
+      updateReaders = true;
+      if (this.closing) {
+        LOG.debug("StoreScanner already closing. There is no need to updateReaders");
+        return;
+      }

We need to close memStoreScanners in StoreScanner#updateReaders before this two return, someone else can take over the task.

Issue Links

• is related to: HBASE-22072 High read/write intensive regions may cause long crash recovery