mcvsubbu commented on issue #7264:
URL: https://github.com/apache/pinot/issues/7264#issuecomment-895513322


   > We don't really have that for case 1 and 2. For case 3, we should not 
expect normal users to check `numServersQueried` and `numServersResponded` and 
compare them to tell whether the result is partial. (The `partialResponse` flag 
is in LinkedIn internal code IIRC)
   
   If partial response flag is not set in all cases, then that is a bug. Let us 
definitely fix it.
   
   I think applications check for exceptions and assume that there is no 
result. So, exceptions when there are partial results may not be the right 
answer. 


-- 
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

Reply via email to