siddharthteotia opened a new pull request #5589: URL: https://github.com/apache/incubator-pinot/pull/5589
## Description Couple of minor fixes: - Both server and broker log write requestId, but they write in a different manner. After looking into the server log for processed queries and then looking into the broker log with corresponding requestId (to look at the actual query) requires `:set ic` and use = instead of : to grep in the broker log with that requestId. A mere copy paste won't work and it wastes time while debugging. This PR makes both server and broker log write requestID in the same manner - On the server side, also log requestDeserialization, responseSerialization times. ## Upgrade Notes Does this PR prevent a zero down-time upgrade? (Assume upgrade order: Controller, Broker, Server, Minion) No Does this PR fix a zero-downtime upgrade introduced earlier? No Does this PR otherwise need attention when creating release notes? Things to consider: No ## Release Notes None ## Documentation N/A ---------------------------------------------------------------- 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: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
