liuml07 commented on code in PR #20935:
URL: https://github.com/apache/flink/pull/20935#discussion_r1022040612


##########
docs/content.zh/docs/deployment/resource-providers/standalone/docker.md:
##########
@@ -405,16 +405,24 @@ services:
   ```
   You can then start creating tables and queries those.
 
-* Note, that all required dependencies (e.g. for connectors) need to be 
available in the cluster as well as the client.
-  For example, if you would like to use the Kafka Connector create a custom 
image with the following Dockerfile
+* Note that all required dependencies (e.g. for connectors) need to be 
available in the cluster as well as the client.
+  For example, if you would like to add and use the SQL Kafka Connector, you 
need to build a custom image.
+  1. Create a Dockerfile named `Kafka.Dockerfile` as follows:
 
   ```Dockerfile
   FROM flink:{{< stable >}}{{< version >}}-scala{{< scala_version >}}{{< 
/stable >}}{{< unstable >}}latest{{< /unstable >}}
-  RUN wget -P /opt/flink/lib 
https://repo.maven.apache.org/maven2/org/apache/flink/flink-sql-connector-kafka_2.12/{{<
 version >}}/flink-sql-connector-kafka_scala{{< scala_version >}}-{{< version 
>}}.jar
+  RUN wget -P /opt/flink/lib 
https://repo.maven.apache.org/maven2/org/apache/flink/flink-sql-connector-kafka/{{<
 version >}}/flink-sql-connector-kafka_scala-{{< version >}}.jar

Review Comment:
   I found the Kafka connector is not including Scala version in the name. So I 
updated this as well.



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