manakbisht opened a new pull request, #20057:
URL: https://github.com/apache/pinot/pull/20057

   ## Problem
   - The retention strategy for hybrid tables can only be configured at the 
cluster level. This is inflexible, as it can’t be enabled selectively per table.
   
   ## Changes
   - Tables can now set their own hybrid retention strategy 
(`segmentsConfig.enableHybridTableRetentionStrategy`). If unset (default), the 
cluster-level config applies, which currently defaults to false.
   - Deprecate the cluster-level config 
(`controller.enable.hybrid.table.retention.strategy`) for removal in a future 
release.
   
   ## Tests
   - Modified `TableConfigSerDeUtilsTest#testSerDe` for 
serialization-deserialization of new field.
   - Added new test 
`RetentionManagerTest#testTableLevelHybridRetentionStrategyOverridesClusterConfig`
 to cover configuration overrides.
   


-- 
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]

Reply via email to