(accumulo) branch 2.1 updated: Does all session clean up for normal scans (#4826)

[kturner]

This is an automated email from the ASF dual-hosted git repository.

kturner pushed a commit to branch 2.1
in repository https://gitbox.apache.org/repos/asf/accumulo.git


The following commit(s) were added to refs/heads/2.1 by this push:
     new d5b476be83 Does all session clean up for normal scans (#4826)
d5b476be83 is described below

commit d5b476be83fc0411bab27c0db0885e508c26e9c0
Author: Keith Turner <ktur...@apache.org>
AuthorDate: Sat Aug 24 09:29:06 2024 -0700

    Does all session clean up for normal scans (#4826)
    
    Normals scans were not running all session cleanup code like batch scans
    were.  AFAICT this change does not impact tablet severs and only scan
    severs.  The code that was not being called only seems to do some clean
    up on scan severs.  Following the code this change should cause
    SnapshotTablet.close() to be called now when a normal scan session is 
cleaned
    up on a scan server.
---
 .../java/org/apache/accumulo/tserver/session/SingleScanSession.java     | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/server/tserver/src/main/java/org/apache/accumulo/tserver/session/SingleScanSession.java
 
b/server/tserver/src/main/java/org/apache/accumulo/tserver/session/SingleScanSession.java
index 0946e259f2..8c48a01263 100644
--- 
a/server/tserver/src/main/java/org/apache/accumulo/tserver/session/SingleScanSession.java
+++ 
b/server/tserver/src/main/java/org/apache/accumulo/tserver/session/SingleScanSession.java
@@ -69,6 +69,6 @@ public class SingleScanSession extends ScanSession {
         ret = true;
       }
     }
-    return ret;
+    return ret && super.cleanup();
   }
 }