Jackie-Jiang commented on issue #10884: URL: https://github.com/apache/pinot/issues/10884#issuecomment-1595412501
Broker request id is generated within the `BrokerRequestHandler`, and I think we can keep it this way, but include it in the `org.apache.pinot.common.response.BrokerResponse`. Seems `org.apache.pinot.client.BrokerResponse` was introduced to avoid dependency on `pinot-common`, but the module already has dependency on `pinot-common` so we may remove it. We need to be cautious about backward compatibility though. @mingmxu Do you want to help contribute this? -- 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]
