jon-wei commented on a change in pull request #10746:
URL: https://github.com/apache/druid/pull/10746#discussion_r556245770
##########
File path:
processing/src/main/java/org/apache/druid/query/QueryUnsupportedException.java
##########
@@ -24,21 +24,20 @@
import com.fasterxml.jackson.annotation.JsonProperty;
import javax.annotation.Nullable;
-import java.net.InetAddress;
/**
* This exception is for the query engine to surface when a query cannot be
run. This can be due to the
* following reasons: 1) The query is not supported yet. 2) The query is not
something Druid would ever supports.
* For these cases, the exact causes and details should also be documented in
Druid user facing documents.
*
- * As a {@link QueryException} it is expected to be serialied to a json
response, but will be mapped to
- * {@link #STATUS_CODE} instead of the default HTTP 500 status.
+ * As a {@link QueryException} it is expected to be serialied to a json
response with a proper HTTP error code
Review comment:
serialied -> serialized
##########
File path:
processing/src/main/java/org/apache/druid/query/ResourceLimitExceededException.java
##########
@@ -19,18 +19,33 @@
package org.apache.druid.query;
+import com.fasterxml.jackson.annotation.JsonCreator;
+import com.fasterxml.jackson.annotation.JsonProperty;
import org.apache.druid.java.util.common.StringUtils;
/**
* Exception indicating that an operation failed because it exceeded some
configured resource limit.
*
- * This is used as a marker exception by {@link QueryInterruptedException} to
report the "Resource limit exceeded"
- * error code.
+ * This is a {@link BadQueryException} because it will likely a user's
misbehavior when this exception is thrown.
+ * Druid cluster operators set some set of resource limitations for some good
reason. When a user query requires
+ * too many resources, it will likely mean that the user query should be
modified to not use excessive resources.
Review comment:
Suggest rewording to something like:
"This is a {@link BadQueryException} because it likely indicates a user's
misbehavior when this exception is thrown. The resource limitations set by
Druid cluster operators are typically less flexible than the parameters of a
user query, so when a user query requires too many resources, the likely remedy
is that the user query should be modified to use fewer resources, or to reduce
query volume."
----------------------------------------------------------------
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...@druid.apache.org
For additional commands, e-mail: commits-h...@druid.apache.org
