raminqaf commented on code in PR #28510:
URL: https://github.com/apache/flink/pull/28510#discussion_r3519728269
##########
flink-table/flink-sql-gateway/src/test/java/org/apache/flink/table/gateway/service/MaterializedTableStatementITCase.java:
##########
@@ -1100,6 +1057,100 @@ void testAlterMaterializedTableAsQueryInFullMode()
throws Exception {
assertThat(oldTable.getDefinitionFreshness()).isEqualTo(newTable.getDefinitionFreshness());
}
+ @Test
+ void
testCreateOrAlterMaterializedTableColumnListReordersSchemaOnCreatePath() throws
Exception {
Review Comment:
I added them after discussing with @snuyanzin about some confusion about
query/schema validation rules applied to MTs. We don't have any tests that
cover these that is why I added them. If you think this will increase the CI
time, I can have them 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]
