Re: [PR] add log with marker for user-based query response exceptions [pinot]

2025-04-08 Thread via GitHub
gortiz commented on code in PR #15433: URL: https://github.com/apache/pinot/pull/15433#discussion_r2032889002 ## pinot-broker/src/main/java/org/apache/pinot/broker/api/resources/PinotClientRequest.java: ## @@ -559,6 +563,16 @@ public static Response getPinotQueryResponse(Broker

Re: [PR] add log with marker for user-based query response exceptions [pinot]

2025-04-08 Thread via GitHub
gortiz commented on code in PR #15433: URL: https://github.com/apache/pinot/pull/15433#discussion_r2032891072 ## pinot-broker/pom.xml: ## @@ -45,6 +45,10 @@ org.apache.pinot pinot-timeseries-planner + + org.apache.logging.log4j + log4j-core +

Re: [PR] add log with marker for user-based query response exceptions [pinot]

2025-04-02 Thread via GitHub
codecov-commenter commented on PR #15433: URL: https://github.com/apache/pinot/pull/15433#issuecomment-2772203727 ## [Codecov](https://app.codecov.io/gh/apache/pinot/pull/15433?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u

[PR] add log with marker for user-based query response exceptions [pinot]

2025-04-02 Thread via GitHub
albertobastos opened a new pull request, #15433: URL: https://github.com/apache/pinot/pull/15433 Adds a log trace whenever the broker includes any exception as part of the query response. This log trace is flagged with a Marker for easier filtering on Logging configuration allowing t