churromorales commented on PR #13498: URL: https://github.com/apache/druid/pull/13498#issuecomment-1340119742
@rohangarg i see your point. If you have a historical give partial data back and disconnect, you will end up re-querying the segments potentially and returning incorrect results. That is no good. I have another potential fix, wanted to go over it with you first though. I think a full query retry is okay in this scenario. On a channel disconnect we could just do a re-query of everything. I think I can add a special flag in the context, and when that gets set the RetryRunner queries the missing intervals, not the missing segments again. What do you think about that? -- 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]
