hongkunxu opened a new pull request, #18665:
URL: https://github.com/apache/pinot/pull/18665
Introduce a user-facing per-query option `enableMaterializedViewRewrite`
(default true) that gates broker-side materialized-view rewrite. The MV minion
query executor injects `SET enableMaterializedViewRewrite='false'` onto its
materialization query so it always reads the base table and is never rewritten
back onto an MV (its own or a sibling), avoiding the circular-rewrite hazard.
Disabling only forgoes an optimization and never changes results. Distinct from
the broker-internal materializedViewRewrite marker, which remains stripped from
user input.
Instructions:
1. The PR has to be tagged with at least one of the following labels (*):
1. `feature`
2. `bugfix`
3. `performance`
4. `ui`
5. `backward-incompat`
6. `release-notes` (**)
2. Remove these instructions before publishing the PR.
(*) Other labels to consider:
- `testing`
- `dependencies`
- `docker`
- `kubernetes`
- `observability`
- `security`
- `code-style`
- `extension-point`
- `refactor`
- `cleanup`
(**) Use `release-notes` label for scenarios like:
- New configuration options
- Deprecation of configurations
- Signature changes to public methods/interfaces
- New plugins added or old plugins removed
--
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]
