This is an automated email from the ASF dual-hosted git repository. taklwu pushed a commit to branch branch-2.5 in repository https://gitbox.apache.org/repos/asf/hbase.git
The following commit(s) were added to refs/heads/branch-2.5 by this push: new 4c1fd0d2e83 HBASE-28055 Performance improvement for scan over several stores. (#5379) 4c1fd0d2e83 is described below commit 4c1fd0d2e83b395debe836fbfc4cfa4cdb337184 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;