[GitHub] [pinot] suddendust edited a comment on issue #7264: Add exception to broker response when not all segments are available (partial response)

[GitBox <gi...@apache.org>]

suddendust edited a comment on issue #7264:
URL: https://github.com/apache/pinot/issues/7264#issuecomment-900126537


   Purely from the user's perspective, this certainly looks like a `warning` than an `error` ("Here are your results but be warned that these are partial"). I think regardless of how we do it in code, a flag indicating this for all the three cases above would be useful. Even now as @Jackie-Jiang pointed out, the user has to check `numServersQueried` and `numServersResponded` and compare them to tell if the result is partial. This is a bit complex unnecessarily. 


-- 
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-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org