Github user GJL commented on a diff in the pull request: https://github.com/apache/flink/pull/4809#discussion_r166001086 --- Diff: docs/ops/state/savepoints.md --- @@ -120,6 +120,10 @@ This will atomically trigger a savepoint for the job with ID `:jobid` and cancel If you don't specify a target directory, you need to have [configured a default directory](#configuration). Otherwise, cancelling the job with a savepoint will fail. +<div class="alert alert-warning"> --- End diff -- Should be something like: ``` <div class="alert alert-warning"> <strong>Attention:</strong> <code>targetDirectory</code> has to be a location accessible by both the JobManager(s) and TaskManager(s), e.g., a location on a distributed file system. </div> ``` because markdown is not rendered within HTML tags. Also, the *warning* you have added is currently in the *Cancel Job with Savepoint* section. I think it should be moved to the section above (*Trigger a Savepoint*), or somewhere else since it does not only apply to the cancelation case. @uce
---