yihua commented on code in PR #8326:
URL: https://github.com/apache/hudi/pull/8326#discussion_r1158041585


##########
hudi-client/hudi-client-common/src/main/java/org/apache/hudi/config/HoodiePayloadConfig.java:
##########
@@ -60,6 +54,14 @@ public class HoodiePayloadConfig extends HoodieConfig {
         + "the record payload class to merge records in the log against each 
other, merge again with the base file and "
         + "produce the final record to be written after compaction.");
 
+  /** @deprecated Use {@link HoodieWriteConfig#PRECOMBINE_FIELD_NAME} and its 
methods instead */

Review Comment:
   The pre-combine field is written to table configs in `hoodie.properties` and 
the read side uses that already.  So is this ok?



-- 
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