jenniferdai opened a new pull request #4632: Revert "Set processingException when all queried segments cannot be a… URL: https://github.com/apache/incubator-pinot/pull/4632 …cquired (#3942)" This reverts commit d4f2ecef660ab1d4efa9696a53b0623aac867c3f. Spoke with Sunitha offline regarding this commit. This PR currently hinges on the state transition OFFLINE -> DROPPED in order to determine segments that were retained out. However, by the time this message is received, the broker has already determined that the server had an error. Retained segments first undergo ONLINE -> OFFLINE then OFFLINE -> DROPPED transition states. Therefore, backing out this PR because it doesn't properly handle retained out segments. In the future, we can change this check to be on the broker side, where we can enforce retention among segments queried.
---------------------------------------------------------------- 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] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
