Jackie-Jiang opened a new pull request, #9648:
URL: https://github.com/apache/pinot/pull/9648

   This PR reverts the special broker negative request id handling for hybrid 
table introduced in #9443, and move the handling to the server side to avoid 
all the logging change related to the request id.
   On the server side, the unique query id is generated as 
`<brokerId>_<requestId>_(O|R)` for OFFLINE and REALTIME request 
correspondingly. For backward compatibility, when trying to cancel a query 
without the type suffix, server will look up both OFFLINE and REALTIME query 
and cancel both of them.
   One behavior change is that when looking up the running queries on the 
server side, the returned query ids will have type suffix. This should be 
desired to reflect the actual queries running.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to