jasperjiaguo commented on code in PR #11942:
URL: https://github.com/apache/pinot/pull/11942#discussion_r1382267013
##########
pinot-core/src/main/java/org/apache/pinot/core/query/scheduler/QueryScheduler.java:
##########
@@ -215,6 +219,14 @@ private byte[] serializeResponse(ServerQueryRequest
queryRequest, InstanceRespon
byte[] responseByte = null;
try {
responseByte = instanceResponse.toDataTable().toBytes();
Review Comment:
Could we verify whether there's memory allocation in toBytes()? We might
need to instrument that part as well. Meanwhile IMO it might be helpful to very
briefly explain why we only added instrumentation only for groupby and
selection operation utils in the PR description @SabrinaZhaozyf
--
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]
