[GitHub] [incubator-pinot] kishoreg commented on pull request #5599: Add Streaming Server Endpoint

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

kishoreg commented on pull request #5599:
URL: https://github.com/apache/incubator-pinot/pull/5599#issuecomment-651230521


   > Right now, we can add logic to the exist broker to accept responses from some of the servers (but not all), and return something to the user with a partial response indication.
   > 
   > Will that still be possible with this mechanism ?
   > 
   > Does each server request have to be sent in its own thread on the broker?
   
   This PR is adding an endpoint to the pinot server. it up to the client (brokers/presto workers) to handle the partial response etc. It will get error/timeout callback. The clients can also cancel the request if it's taking too long.


----------------------------------------------------------------
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:
users@infra.apache.org



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