rohangarg commented on PR #13498: URL: https://github.com/apache/druid/pull/13498#issuecomment-1344628795
@churromorales : Thanks for the context! I skimmed through the implementation and I think you're trying to implement full query retry through the `RetryQueryRunner`. Is that right to say? If so, I had one question regarding it - similar to how broker can consume partial results from historical and then receive a disconnect, it could also happen that the client has also read from the broker some partial results. And after reading some results, a historical disconnects. Now if we retry the whole query, the client may receive duplicate results or some error as well due to truncated results from the first failed try of the query. Is that something that you've considered in your solution? -- 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]
