yashmayya opened a new pull request, #13732: URL: https://github.com/apache/pinot/pull/13732
- Users can use the multi-stage query engine via query options (https://docs.pinot.apache.org/developers/advanced/v2-multi-stage-query-engine) or the (currently undocumented) broker API endpoints `GET /query` and `POST /query` (https://github.com/apache/pinot/pull/11341). - Currently, if the multi-stage engine is disabled via the config `pinot.multistage.engine.enabled` and users issue a broker query request indicating that the multi-stage engine should be used, the query doesn't fail and instead falls back to using the single-stage engine. - This can lead to a confusing user experience. Also, in the controller's SQL query API, we return an error in this scenario - https://github.com/apache/pinot/blob/dd8a6477c22e761a2a6dc3264b9c2a86c94427fd/pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotQueryResource.java#L173-L180 - This patch updates the broker behavior to match the controller behavior when the multi-stage engine is disabled in order to improve and unify the UX. -- 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: commits-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
