kfaraz commented on PR #14916: URL: https://github.com/apache/druid/pull/14916#issuecomment-1701121277
> But it does now seem to me that forceGuaranteedRollup is redundant. If I understand correctly, given a choice of partitioning scheme there is only one possible valid value for forceGuaranteedRollup. Could we simplify things and remove this option? @michaelschiff , yes, I think it would make sense to remove this as a config and just compute the value of the flag based on the partitioning type. -- 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]
