Refactor AbstractMultiTenantReader to remove unnecessary scannerClosed flag and streamline the close method. Update MultiTenantHFileWriter to catch and rethrow exceptions as IOException for improved error handling during section closure.

Author: sairampola

hbase-server/src/main/java/org/apache/hadoop/hbase/io/hfile/AbstractMultiTenantReader.java

@@ -1711,11 +1711,8 @@ public ExtendedCell getNextIndexedKey() {
       return currentScanner.getNextIndexedKey();
     }
 
-    private boolean scannerClosed = false;
-
     @Override
     public void close() {
-      scannerClosed = true;
       try {
         if (currentScanner != null) {
           currentScanner.close();

hbase-server/src/main/java/org/apache/hadoop/hbase/io/hfile/MultiTenantHFileWriter.java

@@ -718,13 +718,13 @@ private void createNewSection(byte[] tenantSectionId, byte[] tenantId) throws IO
       if (customTieringSupplier != null) {
         newSection.setTimeRangeTrackerForTiering(customTieringSupplier);
       }
-    } catch (Exception e) {
+    } catch (RuntimeException e) {
       try {
         newSection.close();
       } catch (IOException closeEx) {
         e.addSuppressed(closeEx);
       }
-      throw e instanceof IOException ? (IOException) e : new IOException(e);
+      throw new IOException(e);
     }
     currentSectionWriter = newSection;