SbloodyS commented on code in PR #16703:
URL:
https://github.com/apache/dolphinscheduler/pull/16703#discussion_r1808248834
##########
docs/docs/en/guide/task/flink.md:
##########
@@ -37,7 +37,7 @@ Flink task type, used to execute Flink programs. For Flink
nodes:
| Parallelism | Used to set the degree of parallelism for
executing Flink tasks.
|
| Yarn queue | Used to set the yarn queue, use `default` queue by
default.
|
| Main program parameters | Set the input parameters for the Flink program and
support the substitution of custom parameter variables.
|
-| Optional parameters | Set the flink command options, such as `-D`, `-C`,
`-yt`.
|
+| Optional parameters | Set the flink command options, such as `-D`, `-C`,
`-yt`, and support the substitution of custom parameter variables, such as
`-Dyarn.application.name=${job_name}` custom parameter job _name will be
replaced. |
Review Comment:
Is this custom parameter `job_name` setting by users with hard code? If so,
this is meaningless. We should add separate field options for users to use
instead of using custom parameters, which are suitable for parameter passing
and global parameters.
--
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]
