vinothchandar commented on code in PR #5627:
URL: https://github.com/apache/hudi/pull/5627#discussion_r909085146
##########
hudi-common/src/main/java/org/apache/hudi/common/table/HoodieTableConfig.java:
##########
@@ -154,6 +155,12 @@ public class HoodieTableConfig extends HoodieConfig {
.withDocumentation("Payload class to use for performing compactions, i.e
merge delta logs with current base file and then "
+ " produce a new base file.");
+ public static final ConfigProperty<String> COMBINE_ENGINE_CLASS_NAME =
ConfigProperty
+ .key("hoodie.compaction.combine.engine.class")
+ .defaultValue(HoodieAvroRecordCombiningEngine.class.getName())
Review Comment:
I updated the ticket. I feel `HoodieMerge` by itself should be designed as
engine aware somehow
--
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]
