davidradl commented on code in PR #26167:
URL: https://github.com/apache/flink/pull/26167#discussion_r1963463378


##########
flink-python/pyflink/table/table.py:
##########
@@ -1077,6 +1078,91 @@ def explain(self, *extra_details: ExplainDetail) -> str:
         j_extra_details = to_j_explain_detail_arr(extra_details)
         return self._j_table.explain(TEXT, j_extra_details)
 
+    def insert_into(
+        self, table_path_or_descriptor: Union[str, TableDescriptor], 
overwrite: bool = False
+    ) -> TablePipeline:
+        """
+        When ``target_path_or_descriptor`` is a table path:
+
+            Declares that the pipeline defined by the given :class:`Table` 
(backed by a
+            DynamicTableSink) should be written to a table that was registered 
under the specified
+            path.
+
+            See the documentation of
+            
:func:`pyflink.table.table_environment.TableEnvironment.use_database` or
+            
:func:`pyflink.table.table_environment.TableEnvironment.use_catalog` for the 
rules on
+            the path resolution.
+
+            Example:
+            ::
+
+                >>> table = table_env.sql_query("SELECTFROM MyTable")
+                >>> table_pipeline = 
table.insert_into_table_path("MySinkTable", True)

Review Comment:
   The function is `Table.insertInto` but the example is a different function 
name `table.insert_into_table_path`. I assumes that  `Table.insertInto` first 
parameter would be Union[str, TableDescriptor] i.e. a string or a table 
descriptor; it would be good to give an example of each. 



-- 
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: issues-unsubscr...@flink.apache.org

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

Reply via email to