aljoscha commented on a change in pull request #12723:
URL: https://github.com/apache/flink/pull/12723#discussion_r442884480
##########
File path: docs/ops/deployment/docker.md
##########
@@ -395,13 +395,13 @@ The next chapters show examples of configuration files to
run Flink.
flink run -d -c ${JOB_CLASS_NAME} /job.jar
```
- * or copy the JAR to the *Flink Master* container and submit the job using
the [CLI](..//cli.html) from there, for example:
+ * or copy the JAR to the *JobManager* container and submit the job using the
[CLI](..//cli.html) from there, for example:
```sh
JOB_CLASS_NAME="com.job.ClassName"
- MASTER_CONTAINER=$(docker ps --filter name=jobmanager --format={% raw
%}{{.ID}}{% endraw %}))
- docker cp path/to/jar "${MASTER_CONTAINER}":/job.jar
- docker exec -t -i "${MASTER_CONTAINER}" flink run -d -c ${JOB_CLASS_NAME}
/job.jar
+ JM_CONTAINER=$(docker ps --filter name=jobmanager --format={% raw
%}{{.ID}}{% endraw %}))
Review comment:
I consider the `JobMaster` an internal component that users rarely see,
though. Or should see.
----------------------------------------------------------------
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]
