gortiz commented on code in PR #14507:
URL: https://github.com/apache/pinot/pull/14507#discussion_r1895389575
##########
pinot-query-runtime/src/main/java/org/apache/pinot/query/runtime/operator/HashJoinOperator.java:
##########
@@ -207,13 +207,17 @@ protected TransferableBlock getNextBlock()
buildBroadcastHashTable();
}
if (_upstreamErrorBlock != null) {
+ LOGGER.trace("Returning upstream error block for join operator");
Review Comment:
Yes, all the changes are just new logs I've added while debuging some
blocking issues. These logs are disabled by default and should be easily
removed by either the jit given they are not allocating. I recommend to keep
them in case we need to debug something in the future.
--
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: commits-unsubscr...@pinot.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org
