yuruguo commented on a change in pull request #14597:
URL: https://github.com/apache/flink/pull/14597#discussion_r554714943
##########
File path:
flink-kubernetes/src/main/java/org/apache/flink/kubernetes/configuration/KubernetesConfigOptions.java
##########
@@ -195,9 +196,16 @@
.stringType()
.defaultValue(getDefaultFlinkImage())
.withDescription(
- "Image to use for Flink containers. "
- + "The specified image must be based upon
the same Apache Flink and Scala versions as used by the application. "
- + "Visit
https://hub.docker.com/_/flink?tab=tags for the official docker images provided
by the Flink project. The Flink project also publishes docker images here:
https://hub.docker.com/r/apache/flink";);
+ Description.builder()
+ .text(
+ "Image to use for Flink
containers. "
+ + "The specified image
must be based upon the same Apache Flink and Scala versions as used by the
application. "
+ + "Visit %s for the
official docker images provided by the Flink project. The Flink project also
publishes docker images to %s.",
+
link("https://hub.docker.com/_/flink?tab=tags";, "here"),
+ link(
+
"https://hub.docker.com/r/apache/flink";,
+ "DockerHub"))
Review comment:
Thanks for you suggestion @wangyang0918, I have updated and submitted
again, please take a look.
----------------------------------------------------------------
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:
[email protected]
