twalthr commented on code in PR #28510:
URL: https://github.com/apache/flink/pull/28510#discussion_r3520002134
##########
flink-table/flink-table-planner/src/test/scala/org/apache/flink/table/planner/plan/stream/sql/TableSinkTest.scala:
##########
@@ -929,6 +929,146 @@ class TableSinkTest extends TableTestBase {
assertEquals(TableTestUtil.replaceStageId(expected),
TableTestUtil.replaceStageId(actual))
}
+ @Test
+ def testExplainCreateMaterializedTable(): Unit = {
Review Comment:
> As a follow-up we can maybe migrate all the Scala Explain related test to
the new Java class.
Yes, for the future feel free to port them when you touch them. Ideally in a
separate commit that builds green. This can be a continuous effort, not a hard
requirement.
--
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]
