Jackie-Jiang opened a new pull request, #15528: URL: https://github.com/apache/pinot/pull/15528
Bugfixes: - Introduce `Enablement` enum with value `ENABLE`, `DISABLE` and `DEFAULT` to control the enablement of a feature. For `DEFAULT` enablement, use the default config from upper level (e.g. instance level) - Introduce `snapshot` and `preload` field as `Enablement` into `UpsertConfig` and `DedupConfig` so that the value can be properly overridden. Currently there is no way to disable at table level when instance level is enabled - Always read properties from `UpsertContext` and `DedupContext` to avoid the inconsistency of server level override and config change Cleanups: - Simplify the constructor for upsert/dedup related configs - Re-order some fields/methods for readability - Unify the metadata manager creation logic for upsert/dedup - Move some constants to `CommonConstants` Incompatible: - Several method signatures are changed Release note: - `enableSnapshot` and `enablePreload` are deprecated and replaced with `snapshot` and `preload` -- 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: commits-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
