[core] Minor refactor arguements for FileStoreCommitImpl

Author: JingsongLi

paimon-core/src/main/java/org/apache/paimon/AbstractFileStore.java

@@ -326,6 +326,7 @@ public FileStoreCommitImpl newCommit(String commitUser, FileStoreTable table) {
                 newStatsFileHandler(),
                 bucketMode(),
                 options.scanManifestParallelism(),
+                createCommitPreCallbacks(table),
                 createCommitCallbacks(commitUser, table),
                 options.commitMaxRetries(),
                 options.commitTimeout(),
@@ -335,8 +336,7 @@ public FileStoreCommitImpl newCommit(String commitUser, FileStoreTable table) {
                 options.commitDiscardDuplicateFiles(),
                 conflictDetectFactory,
                 strictModeChecker,
-                rollback,
-                createCommitPreCallbacks(commitUser, table));
+                rollback);
     }
 
     @Override
@@ -395,6 +395,14 @@ public InternalRowPartitionComputer partitionComputer() {
                 options.legacyPartitionName());
     }
 
+    private List<CommitPreCallback> createCommitPreCallbacks(FileStoreTable table) {
+        List<CommitPreCallback> callbacks = new ArrayList<>();
+        if (options.isChainTable()) {
+            callbacks.add(new ChainTableCommitPreCallback(table));
+        }
+        return callbacks;
+    }
+
     private List<CommitCallback> createCommitCallbacks(String commitUser, FileStoreTable table) {
         List<CommitCallback> callbacks = new ArrayList<>();
 
@@ -434,15 +442,6 @@ private List<CommitCallback> createCommitCallbacks(String commitUser, FileStoreT
         return callbacks;
     }
 
-    private List<CommitPreCallback> createCommitPreCallbacks(
-            String commitUser, FileStoreTable table) {
-        List<CommitPreCallback> callbacks = new ArrayList<>();
-        if (options.isChainTable()) {
-            callbacks.add(new ChainTableCommitPreCallback(table));
-        }
-        return callbacks;
-    }
-
     @Override
     @Nullable
     public PartitionExpire newPartitionExpire(String commitUser, FileStoreTable table) {

paimon-core/src/main/java/org/apache/paimon/operation/FileStoreCommitImpl.java

@@ -150,6 +150,7 @@ public class FileStoreCommitImpl implements FileStoreCommit {
     private final boolean dynamicPartitionOverwrite;
     private final String branchName;
     @Nullable private final Integer manifestReadParallelism;
+    private final List<CommitPreCallback> commitPreCallbacks;
     private final List<CommitCallback> commitCallbacks;
     private final StatsFileHandler statsFileHandler;
     private final BucketMode bucketMode;
@@ -167,8 +168,6 @@ public class FileStoreCommitImpl implements FileStoreCommit {
     private CommitMetrics commitMetrics;
     private boolean appendCommitCheckConflict = false;
 
-    private final List<CommitPreCallback> commitPreCallbacks;
-
     public FileStoreCommitImpl(
             SnapshotCommit snapshotCommit,
             FileIO fileIO,
@@ -193,6 +192,7 @@ public FileStoreCommitImpl(
             StatsFileHandler statsFileHandler,
             BucketMode bucketMode,
             @Nullable Integer manifestReadParallelism,
+            List<CommitPreCallback> commitPreCallbacks,
             List<CommitCallback> commitCallbacks,
             int commitMaxRetries,
             long commitTimeout,
@@ -202,8 +202,7 @@ public FileStoreCommitImpl(
             boolean discardDuplicateFiles,
             ConflictDetection.Factory conflictDetectFactory,
             @Nullable StrictModeChecker strictModeChecker,
-            @Nullable CommitRollback rollback,
-            List<CommitPreCallback> commitPreCallbacks) {
+            @Nullable CommitRollback rollback) {
         this.snapshotCommit = snapshotCommit;
         this.fileIO = fileIO;
         this.schemaManager = schemaManager;
@@ -226,6 +225,7 @@ public FileStoreCommitImpl(
         this.dynamicPartitionOverwrite = dynamicPartitionOverwrite;
         this.branchName = branchName;
         this.manifestReadParallelism = manifestReadParallelism;
+        this.commitPreCallbacks = commitPreCallbacks;
         this.commitCallbacks = commitCallbacks;
         this.commitMaxRetries = commitMaxRetries;
         this.commitTimeout = commitTimeout;
@@ -245,7 +245,6 @@ public FileStoreCommitImpl(
         this.strictModeChecker = strictModeChecker;
         this.conflictDetection = conflictDetectFactory.create(scanner);
         this.commitCleaner = new CommitCleaner(manifestList, manifestFile, indexManifestFile);
-        this.commitPreCallbacks = commitPreCallbacks;
     }
 
     @Override