himanshug commented on pull request #10287: URL: https://github.com/apache/druid/pull/10287#issuecomment-748536451
real numbers reported from #10603 might make this more obvious eventually. That said, On the surface, I do agree with your assessment of the utility of `LogUsedSegments` and understand why you would wanna remove it. That check appears to have existed since the beginning of [druid] time :) and not sure what original motivation was ... maybe there were bugs at the time and adding this check uncovered those and now a days it doesn't flag anything. I would set `druid.coordinator.logUsedSegments.enabled` default value to `false` and leave it undocumented for now. if, after a bunch of releases, nobody notices it then maybe remove that code altogether. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
