kasakrisz opened a new pull request, #5096: URL: https://github.com/apache/hive/pull/5096
### What changes were proposed in this pull request? 1. Materialized view incremental rebuild plans depend on * the MV definitions * and whether the source tables has any delete operation since the last rebuild. Different MV definitions imply different rebuild plans. This PR fixes and improves the first part which establishes the possible incremental rebuild plans based on the MV definition query plan. 2. Reuse this implementation when choosing the final incremental rebuild plan at compiling Alter materialized view rebuild commands ### Why are the changes needed? Show materialized views command printed misleading incremental MV rebuild possibilities when the MV definition has aggregate functions. ### Does this PR introduce _any_ user-facing change? Yes. Output of `SHOW MATERIALIZED VIEWS` change. ### Is the change a dependency upgrade? No. ### How was this patch tested? ``` mvn test -Dtest=TestMaterializedViewIncrementalRewritingRelVisitor -pl ql mvn test -Dtest.output.overwrite -DskipSparkTests -Dtest=TestMiniLlapLocalCliDriver -Dqfile=show_materialized_views.q,materialized_view_create_rewrite.q,materialized_view_create_rewrite_2.q,materialized_view_create_rewrite_3.q,materialized_view_create_rewrite_4.q,materialized_view_create_rewrite_5.q,materialized_view_create_rewrite_6.q,materialized_view_create_rewrite_6_aggr_2joins.q,materialized_view_create_rewrite_6_aggr_3joins.q,materialized_view_create_rewrite_7.q,materialized_view_create_rewrite_8.q,materialized_view_create_rewrite_9.q,materialized_view_create_rewrite_10.q,materialized_view_create_rewrite_11.q,materialized_view_create_rewrite_12.q -pl itests/qtest -Pitests mvn test -Dtest.output.overwrite -Dtest=TestIcebergCliDriver -Dqfile=show_iceberg_materialized_views.q -pl itests/qtest-iceberg -Piceberg -Pitests ``` -- 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]
