shauryachats opened a new pull request, #17161: URL: https://github.com/apache/pinot/pull/17161
## Summary There is a `Preconditions.checkNotNull` check for logical table context within [ServerPlanRequestUtils](https://github.com/apache/pinot/blob/master/pinot-query-runtime/src/main/java/org/apache/pinot/query/runtime/plan/server/ServerPlanRequestUtils.java#L429) which throws a NPE when the server cannot correctly access the logical table context from Helix. When this happens, the `QueryRunner::processQueryBlocking` silently fails with a warn log whereas the broker is stuck waiting for data from the particular server, and times out instead of returning the error message. This change fixes that by including the `OpChain` initialization and compilation inside the `try...catch`. ## Testing Successfully tested by simulating a scenario where logical table context was inaccessible for a particular server. Error before fix: ``` ``` Error after fix: ``` {"exceptions":[{"errorCode":1000,"message":"Received 1 error from stage 5 on Server_localhost_32001: LogicalTableContext not found for logical table name: logical_table_2, query context id: 639209137000000000"}]} ``` -- 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]
