SbloodyS commented on code in PR #16703:
URL:
https://github.com/apache/dolphinscheduler/pull/16703#discussion_r1808361488
##########
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:
As I said that Custom parameters are suitable for parameter passing between
tasks in a workflow and global parameters. `yarn.application.name` is a
parameter of task level with different functions. We should not add it here.
--
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]
