This is an automated email from the ASF dual-hosted git repository.
gortiz pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git
The following commit(s) were added to refs/heads/master by this push:
new 9be162a8b4 Include exception when no server is found (#14709)
9be162a8b4 is described below
commit 9be162a8b4c9d956da1c735711c968e5bd2921fc
Author: Gonzalo Ortiz Jaureguizar <[email protected]>
AuthorDate: Mon Dec 23 14:20:01 2024 +0100
Include exception when no server is found (#14709)
---
.../broker/requesthandler/BaseSingleStageBrokerRequestHandler.java | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git
a/pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseSingleStageBrokerRequestHandler.java
b/pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseSingleStageBrokerRequestHandler.java
index 1364919592..b031e47617 100644
---
a/pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseSingleStageBrokerRequestHandler.java
+++
b/pinot-broker/src/main/java/org/apache/pinot/broker/requesthandler/BaseSingleStageBrokerRequestHandler.java
@@ -703,7 +703,10 @@ public abstract class BaseSingleStageBrokerRequestHandler
extends BaseBrokerRequ
if (offlineBrokerRequest == null && realtimeBrokerRequest == null) {
if (!exceptions.isEmpty()) {
- LOGGER.info("No server found for request {}: {}", requestId, query);
+ ProcessingException firstException = exceptions.get(0);
+ String logTail = exceptions.size() > 1 ? (exceptions.size()) + "
exceptions found. Logging only the first one"
+ : "1 exception found";
+ LOGGER.info("No server found for request {}: {}. {}", requestId,
query, logTail, firstException);
_brokerMetrics.addMeteredTableValue(rawTableName,
BrokerMeter.NO_SERVER_FOUND_EXCEPTIONS, 1);
return new BrokerResponseNative(exceptions);
} else {
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]