[hbase] branch branch-2.4 updated: HBASE-28055 Performance improvement for scan over several stores. (#5379)

[taklwu]

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

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


The following commit(s) were added to refs/heads/branch-2.4 by this push:
     new 62a6a2923af HBASE-28055 Performance improvement for scan over several 
stores. (#5379)
62a6a2923af is described below

commit 62a6a2923af89fa1cef960bf3c1c458d6e6fe2f7
Author: Sergey Soldatov <ssolda...@cloudera.com>
AuthorDate: Tue Sep 5 16:59:28 2023 -0700

    HBASE-28055 Performance improvement for scan over several stores. (#5379)
    
    Signed-off-by: Ankit Singhal <an...@apache.org>
    Signed-off-by: Tak Lon (Stephen) Wu <tak...@apache.org>
---
 .../main/java/org/apache/hadoop/hbase/regionserver/StoreScanner.java  | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git 
a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/StoreScanner.java
 
b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/StoreScanner.java
index bf95289440f..2a4dcb169c5 100644
--- 
a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/StoreScanner.java
+++ 
b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/StoreScanner.java
@@ -913,7 +913,9 @@ public class StoreScanner extends 
NonReversedNonLazyKeyValueScanner
     // We need this check because it may happen that the new scanner that we 
get
     // during heap.next() is requiring reseek due of fake KV previously 
generated for
     // ROWCOL bloom filter optimization. See HBASE-19863 for more details
-    if (useRowColBloom && nextCell != null && 
matcher.compareKeyForNextColumn(nextCell, cell) < 0) {
+    if (
+      useRowColBloom && nextCell != null && cell.getTimestamp() == 
PrivateConstants.OLDEST_TIMESTAMP
+    ) {
       return false;
     }
     return true;