yihua commented on code in PR #11943:
URL: https://github.com/apache/hudi/pull/11943#discussion_r1811149181
##########
hudi-common/src/main/java/org/apache/hudi/common/model/RecordPayloadType.java:
##########
@@ -94,19 +94,17 @@ public static RecordPayloadType fromClassName(String
className) {
return CUSTOM;
}
- public static String getPayloadClassName(HoodieConfig config) {
+ public static Option<String> getPayloadClassName(HoodieConfig config) {
String payloadClassName;
if (config.contains(PAYLOAD_CLASS_NAME)) {
payloadClassName = config.getString(PAYLOAD_CLASS_NAME);
- } else if (config.contains(PAYLOAD_TYPE)) {
- payloadClassName =
RecordPayloadType.valueOf(config.getString(PAYLOAD_TYPE)).getClassName();
Review Comment:
Based on our discussion, the beta releases are not considered
production-ready, so any new configs added in the beta releases only can be
removed without considering backwards compatibility.
--
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]
