gortiz opened a new pull request, #15534: URL: https://github.com/apache/pinot/pull/15534
This PR removes a problematic check when inferPartitionHint is true. The original idea was to fail when the worker manager is null in a broker and this query option is true. However, this situation is not possible anymore given that brokers always have a worker manager. Even worse, this constraint always failed when the query was executed through a controller, given that they needed to parse the query to get the tables being used. At the same time, they never set the worker manager. Therefore, by removing this check, we: * Do not affect brokers * Fix an error in controllres. -- 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]
