ankitsultana commented on PR #10706: URL: https://github.com/apache/pinot/pull/10706#issuecomment-1535191689
> @ankitsultana I have gone with `(host:port__requestID)`. Does that work or do you want something else? I think we should use broker instance id. In containerized envs host:port doesn't have much meaning. @KKcorps : To unblock you and also allow revisiting this in the future, let's return a string requestId (which may simply be `Long.toString(requestId)` for now). Once we change the requestId everywhere, we can change the broker response also accordingly. This is also consistent with the fact that the current requestId is a long and the returned response from Broker should match with whatever ID the broker uses. -- 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]
