wenlong88 commented on a change in pull request #14428:
URL: https://github.com/apache/flink/pull/14428#discussion_r546598135



##########
File path: 
flink-table/flink-table-planner-blink/src/test/resources/explain/testStatementSetExecutionExplain.out
##########
@@ -32,9 +32,9 @@ 
LegacySink(name=[`default_catalog`.`default_database`.`MySink`], fields=[first])
     } ]
   }, {
     "id" : ,
-    "type" : 
"SourceConversion(table=[default_catalog.default_database.MyTable, source: 
[CsvTableSource(read fields: last)]], fields=[last])",
+    "type" : "SourceConversionToRowData",
     "pact" : "Operator",
-    "contents" : 
"SourceConversion(table=[default_catalog.default_database.MyTable, source: 
[CsvTableSource(read fields: last)]], fields=[last])",
+    "contents" : "SourceConversionToRowData",

Review comment:
       the same as the pr before, it would be better to have the info of table 
in operator name.




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

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to