You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by cc...@apache.org on 2026/10/13 17:35:27 UTC

(hbase) branch master updated: Inefficient HFile format check in incremental backup bulkload copy (#8710)

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

charlesconnell 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 a3de15b260c Inefficient HFile format check in incremental backup bulkload copy (#8710)
a3de15b260c is described below

commit a3de15b260cdfb45642e18414afff18205744e02
Author: tanya stickles <ta...@gmail.com>
AuthorDate: Tue Oct 13 13:35:19 2026 -0400

    Inefficient HFile format check in incremental backup bulkload copy (#8710)
    
    Co-authored-by: Tanya Stickles <ts...@hubspot.com>
---
 .../apache/hadoop/hbase/backup/impl/IncrementalTableBackupClient.java | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/hbase-backup/src/main/java/org/apache/hadoop/hbase/backup/impl/IncrementalTableBackupClient.java b/hbase-backup/src/main/java/org/apache/hadoop/hbase/backup/impl/IncrementalTableBackupClient.java
index 0e11c0f1289..9d3ba570766 100644
--- a/hbase-backup/src/main/java/org/apache/hadoop/hbase/backup/impl/IncrementalTableBackupClient.java
+++ b/hbase-backup/src/main/java/org/apache/hadoop/hbase/backup/impl/IncrementalTableBackupClient.java
@@ -53,10 +53,10 @@ import org.apache.hadoop.hbase.backup.util.BackupUtils;
 import org.apache.hadoop.hbase.client.Admin;
 import org.apache.hadoop.hbase.client.ColumnFamilyDescriptor;
 import org.apache.hadoop.hbase.client.Connection;
-import org.apache.hadoop.hbase.io.hfile.HFile;
 import org.apache.hadoop.hbase.mapreduce.HFileOutputFormat2;
 import org.apache.hadoop.hbase.mapreduce.WALInputFormat;
 import org.apache.hadoop.hbase.mapreduce.WALPlayer;
+import org.apache.hadoop.hbase.regionserver.StoreFileInfo;
 import org.apache.hadoop.hbase.snapshot.SnapshotDescriptionUtils;
 import org.apache.hadoop.hbase.snapshot.SnapshotManifest;
 import org.apache.hadoop.hbase.snapshot.SnapshotRegionLocator;
@@ -614,7 +614,7 @@ public class IncrementalTableBackupClient extends TableBackupClient {
         List<String> files = new ArrayList<>();
         while (locatedFiles.hasNext()) {
           LocatedFileStatus file = locatedFiles.next();
-          if (file.isFile() && HFile.isHFileFormat(tgtFs, file.getPath())) {
+          if (file.isFile() && StoreFileInfo.isHFile(file.getPath())) {
             files.add(file.getPath().toString());
           }
         }