jasperjiaguo commented on code in PR #10683:
URL: https://github.com/apache/pinot/pull/10683#discussion_r1190134455
##########
pinot-common/src/main/java/org/apache/pinot/common/exception/QueryException.java:
##########
@@ -54,7 +54,7 @@ public static void setMaxLinesOfStackTrace(int
maxLinesOfStackTracePerFrame) {
public static final int ACCESS_DENIED_ERROR_CODE = 180;
public static final int TABLE_DOES_NOT_EXIST_ERROR_CODE = 190;
public static final int QUERY_EXECUTION_ERROR_CODE = 200;
- public static final int QUERY_CANCELLATION_ERROR_CODE = 205;
+ public static final int QUERY_CANCELLATION_ERROR_CODE = 503;
Review Comment:
As the error is returned from the runner thread, not worker threads through
the result block, having the error code test in
OfflineClusterMemBasedServerQueryKilingTest and
OfflineClusterMemBasedBrokerQueryKilingTest would suffice
--
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
