dianfu commented on a change in pull request #9370: [FLINK-13594][python] 
Improve the 'from_element' method of flink python api to apply to blink planner
URL: https://github.com/apache/flink/pull/9370#discussion_r311344959
 
 

 ##########
 File path: 
flink-table/flink-table-planner/src/main/scala/org/apache/flink/table/util/python/PythonTableUtils.scala
 ##########
 @@ -422,3 +398,23 @@ object PythonTableUtils {
     result
   }
 }
+
+/**
+  * An InputFormatTableSource created by python 'from_element' method.
+  *
+  * @param inputFormat The input format which contains the python data 
collection,
+  *                    usually created by PythonTableUtils#getInputFormat 
method
+  * @param rowTypeInfo The row type info of the python data.
+  *                    It is generated by the python 'from_element' method.
+  */
+class PythonCollectionInputFormatTableSource[Row](
+    inputFormat: InputFormat[Row, _ <: InputSplit],
 
 Review comment:
   Good point. What about rename the class name to 
PythonInputFormatTableSource? It allows to change the inputFormat 
implementation in the future. 

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


With regards,
Apache Git Services

Reply via email to