wombatu-kun commented on code in PR #18375:
URL: https://github.com/apache/hudi/pull/18375#discussion_r3198653909


##########
hudi-client/hudi-client-common/src/main/java/org/apache/hudi/config/HoodieWriteConfig.java:
##########
@@ -3756,6 +3761,43 @@ protected void setDefaults() {
       autoAdjustConfigsForConcurrencyMode(isLockProviderPropertySet);
     }
 
+    // Class name referenced by string so hudi-client-common does not depend 
on hudi-spark-client.
+    private static final String DEFAULT_SPARK_RECORD_MERGER_CLASS = 
"org.apache.hudi.DefaultSparkRecordMerger";

Review Comment:
   Verified at HoodieWriteConfig.java:3098 — 
`DEFAULT_SPARK_RECORD_MERGER_CLASS` sits at the top of the `Builder` class 
(line 3095) with the other field declarations. Pushed at 2def3286, please 
resolve when convenient.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to