(amoro) branch master updated: [Improvement] Remove "arctic.enable" for production files (#2724)

[jinsongzhou]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 84e095b6e [Improvement] Remove "arctic.enable" for production files 
(#2724)
84e095b6e is described below

commit 84e095b6e28a1730d2709daedf0acbffad7b12fc
Author: Xie Yi <105206850+xieyi...@users.noreply.github.com>
AuthorDate: Thu Apr 11 16:04:21 2024 +0800

    [Improvement] Remove "arctic.enable" for production files (#2724)
    
    remove ARCTIC_TABLE_FLAG_LEGACY
---
 .../java/com/netease/arctic/hive/utils/HiveMetaSynchronizer.java    | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git 
a/mixed-format/hive/src/main/java/com/netease/arctic/hive/utils/HiveMetaSynchronizer.java
 
b/mixed-format/hive/src/main/java/com/netease/arctic/hive/utils/HiveMetaSynchronizer.java
index 5e4f737bc..1327408dc 100644
--- 
a/mixed-format/hive/src/main/java/com/netease/arctic/hive/utils/HiveMetaSynchronizer.java
+++ 
b/mixed-format/hive/src/main/java/com/netease/arctic/hive/utils/HiveMetaSynchronizer.java
@@ -241,10 +241,8 @@ public class HiveMetaSynchronizer {
               filesToDelete.addAll(filesGroupedByPartition.get(partitionData));
               filesToAdd.addAll(hiveDataFiles);
               // make sure new partition is not created by arctic
-            } else if 
(hivePartition.getParameters().get(HiveTableProperties.ARCTIC_TABLE_FLAG)
-                    == null
-                && 
hivePartition.getParameters().get(HiveTableProperties.ARCTIC_TABLE_FLAG_LEGACY)
-                    == null) {
+            } else if (!CompatibleHivePropertyUtil.propertyAsBoolean(
+                hivePartition.getParameters(), 
HiveTableProperties.ARCTIC_TABLE_FLAG, false)) {
               filesToAdd.addAll(hiveDataFiles);
             }
           }