snuyanzin commented on code in PR #28510:
URL: https://github.com/apache/flink/pull/28510#discussion_r3510765497


##########
flink-table/flink-table-planner/src/test/scala/org/apache/flink/table/planner/plan/stream/sql/TableSinkTest.scala:
##########
@@ -929,6 +929,137 @@ class TableSinkTest extends TableTestBase {
     assertEquals(TableTestUtil.replaceStageId(expected), 
TableTestUtil.replaceStageId(actual))
   }
 
+  @Test
+  def testExplainCreateMaterializedTable(): Unit = {
+    val actual = util.tableEnv.explainSql("""
+                                            |CREATE MATERIALIZED TABLE 
MyMTTable
+                                            | WITH (
+                                            |   'connector' = 'values'
+                                            |) AS
+                                            |  SELECT
+                                            |    `a`,
+                                            |    `b`
+                                            |  FROM
+                                            |    MyTable
+                                            |""".stripMargin)
+
+    val expected = 
TableTestUtil.readFromResource("/explain/testExplainCreateMT.out")
+
+    assertEquals(TableTestUtil.replaceStageId(expected), 
TableTestUtil.replaceStageId(actual))
+  }
+
+  @Test
+  def testExplainCreateOrAlterMaterializedTable(): Unit = {
+    val actual = util.tableEnv.explainSql("""
+                                            |CREATE OR ALTER MATERIALIZED 
TABLE MyMTTable
+                                            | WITH (
+                                            |   'connector' = 'values'
+                                            |) AS
+                                            |  SELECT
+                                            |    `a`,
+                                            |    `b`
+                                            |  FROM
+                                            |    MyTable
+                                            |""".stripMargin)
+
+    val expected = 
TableTestUtil.readFromResource("/explain/testExplainCreateOrAlterMT.out")
+
+    assertEquals(TableTestUtil.replaceStageId(expected), 
TableTestUtil.replaceStageId(actual))
+  }
+
+  @Test
+  def testExplainAlterMaterializedTable(): Unit = {
+    util.addTable(s"""
+                     |CREATE OR ALTER MATERIALIZED TABLE MyMTTable
+                     | WITH (
+                     |   'connector' = 'values'
+                     |) AS
+                     |  SELECT
+                     |    `a`,
+                     |    `b`
+                     |  FROM
+                     |    MyTable
+                     |""".stripMargin)
+    val actual = util.tableEnv.explainSql("""
+                                            |ALTER MATERIALIZED TABLE MyMTTable
+                                            |AS
+                                            |  SELECT
+                                            |    `a`,
+                                            |    `b`
+                                            |  FROM
+                                            |    MyTable
+                                            |""".stripMargin)
+
+    val expected = 
TableTestUtil.readFromResource("/explain/testExplainAlterMT.out")

Review Comment:
   strange approach: why does it not take into account current method name?
   
   however seems present in master already...



-- 
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]

Reply via email to