klsince commented on code in PR #15268:
URL: https://github.com/apache/pinot/pull/15268#discussion_r1996345949
##########
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/dedup/TableDedupMetadataManagerFactory.java:
##########
@@ -39,6 +39,9 @@ private TableDedupMetadataManagerFactory() {
public static final String DEDUP_DEFAULT_METADATA_MANAGER_CLASS =
"default.metadata.manager.class";
public static final String DEDUP_DEFAULT_ENABLE_PRELOAD =
"default.enable.preload";
+ public static final String
DEDUP_DEFAULT_ALLOW_PARTIAL_DEDUP_CONSUMPTION_DURING_COMMIT =
+ "default.allow.partial.dedup.consumption.during.commit";
Review Comment:
remove `partial` from var name and the config name
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
