1996fanrui commented on code in PR #26341:
URL: https://github.com/apache/flink/pull/26341#discussion_r2013473151


##########
docs/content.zh/docs/dev/table/tableApi.md:
##########
@@ -363,7 +363,7 @@ Table result = orders.select($("a"), $("c").as("d"));
 {{< tab "Scala" >}}
 ```scala
 val orders = tableEnv.from("Orders")
-Table result = orders.select($"a", $"c" as "d")
+val result = orders.select($"a", $"c" as "d")

Review Comment:
   > Although the Scala API has been removed, Scala can still write Flink 
programs using the Java API.
   
   If I understand correctly, it works because the Scala language is compatible 
with Java language. And Flink community doesn't promise anything about it in 
flink v2. 
   
   Also, as the Proposed Changes part mentioned in the FLIP-265 [1] : `We 
remove all Scala API documentation`. So I guess that removing scala-related 
document in table api page[2] is missed.
   
   I'd like to ping @MartijnVisser to confirm it, he is the initiator of 
FLIP-265.
   
   
   [1] 
https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=231115782#FLIP265DeprecateandremoveScalaAPIsupport-ProposedChanges
   [2] 
https://nightlies.apache.org/flink/flink-docs-release-2.0/docs/dev/table/tableapi/



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