jihoonson commented on pull request #12250: URL: https://github.com/apache/druid/pull/12250#issuecomment-1042098496
> Does this really need to be behind a featureFlag? That seems like it's just adding config for not too much value? What's the feature flag trying to protect from, especially if it defaults to true? @imply-cheddar I added it just in case where there is a bug in my code. I think its coverage is quite good, but I don't trust my code in general :slightly_smiling_face: I want to avoid an extreme case where people cannot read their segments because of some bug I introduced and have to downgrade their cluster to recreate their segments. With having a feature flag, they can just recreate segments with no downgrade. Code-wise, the feature flag doesn't add much code as most of them are needed to not support Hadoop and Tranquility. We may add this feature for Hadoop in the future, but I'm sure we won't do it for Tranquility. -- 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]
