tibrewalpratik17 opened a new pull request, #14406: URL: https://github.com/apache/pinot/pull/14406
Currently, we skip deepstore upload retry task in controller for tables enabled with no retention config. We have configured multiple tables in Uber with no retention config and rely on upsert-compaction to cleanup empty segments (segments with no valid records). Having no deepstore upload retry for these segments poses 2 issues: - Reliability issues during server restarts. - Upsert compaction is also skipped for these segments as we have a condition in generator to check for empty ZK metadata URL (Will enhance the logic here as well in another PR). This PR fixes the issue to allow deepstore upload retry for infinite retention tables. -- 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]
