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

pwason pushed a commit to branch release-0.14.0
in repository https://gitbox.apache.org/repos/asf/hudi.git

commit 2aaf4027110d40b719a62c4bda74d9453f22f22f
Author: Sivabalan Narayanan <[email protected]>
AuthorDate: Wed Aug 30 11:48:48 2023 -0400

    [MINOR] Fixing warn log with auto key gen (#9547)
---
 .../main/scala/org/apache/hudi/AutoRecordKeyGenerationUtils.scala  | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git 
a/hudi-spark-datasource/hudi-spark-common/src/main/scala/org/apache/hudi/AutoRecordKeyGenerationUtils.scala
 
b/hudi-spark-datasource/hudi-spark-common/src/main/scala/org/apache/hudi/AutoRecordKeyGenerationUtils.scala
index 501c563a989..6c1b828f3be 100644
--- 
a/hudi-spark-datasource/hudi-spark-common/src/main/scala/org/apache/hudi/AutoRecordKeyGenerationUtils.scala
+++ 
b/hudi-spark-datasource/hudi-spark-common/src/main/scala/org/apache/hudi/AutoRecordKeyGenerationUtils.scala
@@ -48,10 +48,9 @@ object AutoRecordKeyGenerationUtils {
       if (!parameters.getOrElse(HoodieTableConfig.POPULATE_META_FIELDS.key(), 
HoodieTableConfig.POPULATE_META_FIELDS.defaultValue().toString).toBoolean) {
         throw new HoodieKeyGeneratorException("Disabling " + 
HoodieTableConfig.POPULATE_META_FIELDS.key() + " is not supported with auto 
generation of record keys")
       }
-    }
-
-    if (hoodieConfig.contains(PRECOMBINE_FIELD.key())) {
-      log.warn("Precombine field " + 
hoodieConfig.getString(PRECOMBINE_FIELD.key()) + " will be ignored with auto 
record key generation enabled")
+      if (hoodieConfig.contains(PRECOMBINE_FIELD.key())) {
+        log.warn("Precombine field " + 
hoodieConfig.getString(PRECOMBINE_FIELD.key()) + " will be ignored with auto 
record key generation enabled")
+      }
     }
   }
 

Reply via email to