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

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


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

commit 33e7e5315b4244196e1652069bf7f8662103969c
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 2b6ac583ff6..c12307841a2 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
@@ -935,7 +935,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;

Reply via email to