You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by zh...@apache.org on 2026/02/25 14:37:26 UTC

(hbase) branch branch-2 updated: HBASE-29874 Removing the useless parameters from ScanQueryMatcher.isCellTTLExpired (#7720)

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

zhangduo pushed a commit to branch branch-2
in repository https://gitbox.apache.org/repos/asf/hbase.git


The following commit(s) were added to refs/heads/branch-2 by this push:
     new 11d4ac57bba HBASE-29874 Removing the useless parameters from ScanQueryMatcher.isCellTTLExpired (#7720)
11d4ac57bba is described below

commit 11d4ac57bba737e577134d45d4ad3d266493a14f
Author: Liu Xiao <42...@users.noreply.github.com>
AuthorDate: Wed Feb 25 22:17:18 2026 +0800

    HBASE-29874 Removing the useless parameters from ScanQueryMatcher.isCellTTLExpired (#7720)
    
    Signed-off-by: Duo Zhang <zh...@apache.org>
    Signed-off-by: Junegunn Choi <ju...@apache.org>
    (cherry picked from commit e2cfd2c8cfa7689360b161c6f4a7fa3829ed5685)
---
 .../hadoop/hbase/regionserver/querymatcher/ScanQueryMatcher.java     | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/querymatcher/ScanQueryMatcher.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/querymatcher/ScanQueryMatcher.java
index f90feda2957..b5236bd1f2d 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/querymatcher/ScanQueryMatcher.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/querymatcher/ScanQueryMatcher.java
@@ -139,8 +139,7 @@ public abstract class ScanQueryMatcher implements ShipperListener {
   }
 
   /** Returns true if the cell is expired */
-  private static boolean isCellTTLExpired(final ExtendedCell cell, final long oldestTimestamp,
-    final long now) {
+  private static boolean isCellTTLExpired(final ExtendedCell cell, final long now) {
     // Look for a TTL tag first. Use it instead of the family setting if
     // found. If a cell has multiple TTLs, resolve the conflict by using the
     // first tag encountered.
@@ -175,7 +174,7 @@ public abstract class ScanQueryMatcher implements ShipperListener {
     }
 
     // check if the cell is expired by cell TTL
-    if (isCellTTLExpired(cell, this.oldestUnexpiredTS, this.now)) {
+    if (isCellTTLExpired(cell, this.now)) {
       return MatchCode.SKIP;
     }