gortiz commented on code in PR #15037:
URL: https://github.com/apache/pinot/pull/15037#discussion_r1973618179
##########
pinot-common/src/main/java/org/apache/pinot/common/response/broker/BrokerQueryErrorMessage.java:
##########
@@ -18,24 +18,39 @@
*/
package org.apache.pinot.common.response.broker;
+import com.fasterxml.jackson.annotation.JsonCreator;
import com.fasterxml.jackson.annotation.JsonProperty;
+import org.apache.pinot.spi.exception.QueryErrorCode;
+import org.apache.pinot.spi.exception.QueryErrorMessage;
/**
* This class represents an exception using a message and an error code.
+ *
+ * This is only used to serialize the error message and error code when a
broker sends an error message to the client.
+ * In other cases use {@link QueryErrorMessage} instead.
*/
-public class QueryProcessingException {
+public class BrokerQueryErrorMessage {
Review Comment:
I've just pushed a commit changing the name to `BrokerResponseErrorMessage`.
--
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
