suddendust commented on issue #7264: URL: https://github.com/apache/pinot/issues/7264#issuecomment-897426863
I think the fundamental concern here is that partial results might be returned without the users taking cognisance of them. While having a flag like `partialResponse` solves to an extent, it is still prone to being missed by the user (many users don't even look at the response stats if the query was fast enough). I agree with @yupeng9 that the default behaviour in such cases should be failing the query. If the user has explicitly specified that he is okay with partial responses, then it can be indicated in a flag to the user. So this entails three changes: 1. Throwing exception from the broker in all the three cases. 2. Indicating the user about partial response in all the three cases. 3. Changing the default query behaviour. -- 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: commits-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
