Github user uce commented on a diff in the pull request:

    https://github.com/apache/flink/pull/2609#discussion_r82980824
  
    --- Diff: docs/setup/cli.md ---
    @@ -113,6 +113,10 @@ The command line can be used to
     
             ./bin/flink cancel <jobID>
     
    +-   Cancel a job with a savepoint:
    +
    +        ./bin/flink cancel -s [targetDirectory] <jobID>
    --- End diff --
    
    We could also change the savepoint trigger command to `savepoint -t 
[targetDir] <jobID>`.
    
    Or we could allow `cancel <jobID> [targetDirectory]` in addition to the 
`-s` option variant, but that might be slightly confusing, too. We need the 
`-s` option without an argument for cancel with savepoint to the default target.
    
    What do you think?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to