linliu-code commented on code in PR #13616:
URL: https://github.com/apache/hudi/pull/13616#discussion_r2229862724
##########
hudi-common/src/main/java/org/apache/hudi/common/table/HoodieTableConfig.java:
##########
@@ -321,14 +321,13 @@ public class HoodieTableConfig extends HoodieConfig {
.withDocumentation("This property when set, will define how two versions
of the record will be "
+ "merged together where the later contains only partial set of
values and not entire record.");
- public static final ConfigProperty<String> MERGE_PROPERTIES = ConfigProperty
- .key("hoodie.table.merge.properties")
+ public static final ConfigProperty<String> MERGE_CUSTOM_PROPERTY_PREFIX =
ConfigProperty
+ .key("hoodie.merge.custom.property.prefix")
.noDefaultValue()
.sinceVersion("1.1.0")
- .withDocumentation("The value of this property is in the format of
'K1=V1,K2=V2,...,Ki=Vi,...'. "
- + "Each (Ki, Vi) pair represents a property used during merge
scenarios. "
- + "Some merge mode might need some writer properties that are
required for readers to function as expected. "
- + "Hudi stores those properties here so readers can set them while
reading.");
+ .withDocumentation("Some merge mode might need writer properties that
are required for readers "
Review Comment:
I modified the comments for this prefix constant.
--
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]
