yashmayya opened a new pull request, #13977: URL: https://github.com/apache/pinot/pull/13977
- Currently, query null handling support can be enabled via the query option `enableNullHandling` (see https://docs.pinot.apache.org/developers/advanced/null-value-support#advanced-null-handling-support). By default, this "advanced" null handling support is disabled for both the v1 and v2 query engines. - This patch adds a new broker config `pinot.broker.query.enable.null.handling` that will allow users to enable null handling for queries by default (i.e., without having to set the query option for every query). Note that this applies to both the query engines. - This is useful if users always want null handling to be enabled especially in cases where the queries are being generated by an external system where the user doesn't have any control over the individual queries (and hence can't set the query option explicitly). - If needed, users can still override the null handling behavior on a per query basis using the existing query option even when the new broker config is set. - Note that even if we enable null handling support by default for the multistage query engine in the future (see https://github.com/apache/pinot/pull/13570), the behavior of this broker config won't change - i.e., if it is explicitly configured to `false` this will disable query null handling support for both the query engines. -- 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
