amrishlal commented on a change in pull request #6998:
URL: https://github.com/apache/incubator-pinot/pull/6998#discussion_r656758757



##########
File path: 
pinot-core/src/main/java/org/apache/pinot/core/operator/combine/BaseCombineOperator.java
##########
@@ -163,7 +163,7 @@ protected void processSegments(int taskIndex) {
         LOGGER.error("Caught exception while executing operator of index: {} 
(query: {})", operatorIndex, _queryContext,
             e);
         _blockingQueue.offer(new IntermediateResultsBlock(e));
-        return;
+        throw e;

Review comment:
       Reverting this change as I am no longer able to reproduce the case. But 
if I recall correctly, I was seeing exceptions thrown in JSON_EXTRACT_SCALAR 
getting eaten up and not showing up in QueryConsole while running under 
QuickStart. If it reappears it should be easy to catch.




-- 
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:
us...@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org

Reply via email to