yashmayya commented on code in PR #16076:
URL: https://github.com/apache/pinot/pull/16076#discussion_r2144530241
##########
pinot-query-runtime/src/main/java/org/apache/pinot/query/mailbox/ReceivingMailbox.java:
##########
@@ -61,8 +61,8 @@ public class ReceivingMailbox {
public static final int DEFAULT_MAX_PENDING_BLOCKS = 5;
private static final Logger LOGGER =
LoggerFactory.getLogger(ReceivingMailbox.class);
- private static final MseBlockWithStats CANCELLED_ERROR_BLOCK = new
MseBlockWithStats(
- ErrorMseBlock.fromException(new RuntimeException("Cancelled by
receiver")), Collections.emptyList());
+ // This was previously a static fina; attribute, but now that includes
server and stage, we cannot use constants
Review Comment:
```suggestion
// This was previously a static final attribute, but now that includes
server and stage, we cannot use constants
```
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
