xiangfu0 commented on PR #18242: URL: https://github.com/apache/pinot/pull/18242#issuecomment-4539569262
Closing — this combined PR has been split into three smaller, focused PRs that have either landed or are tracked separately: 1. **#18528** "[feature] SSE Materialized View — view creation, ingestion (PR 1 of 2)" — **merged**. 2. **#18564** "Add isMaterializedView flag on TableConfig as the single MV identity" — **merged**. 3. **#18529** "[feature] SSE Materialized View — broker query rewrite (PR 2 of 2)" — **open / under review**; all CI checks green on the current head. A rebase attempt of this branch onto current `upstream/master` produced only `add/add` conflicts on the MV files that #18528 already brought in plus the broker-rewrite files that #18529 already contains in their reviewed form. The only file unique to this branch (`MaterializedViewRewriteResult.java`) was intentionally collapsed during #18529's review iterations — it was a single-field wrapper around a nullable plan, replaced by returning the plan directly. Nothing else from this branch is missing from `master + #18529`. Please follow #18529 for the remaining work. -- 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]
