You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by GitBox <gi...@apache.org> on 2019/07/09 12:33:58 UTC
[GitHub] [flink] kl0u commented on a change in pull request #9030:
[FLINK-13123] Align Stop/Cancel Commands in CLI and REST Interface and
Improve Documentation
kl0u commented on a change in pull request #9030: [FLINK-13123] Align Stop/Cancel Commands in CLI and REST Interface and Improve Documentation
URL: https://github.com/apache/flink/pull/9030#discussion_r301558502
##########
File path: flink-runtime/src/main/java/org/apache/flink/runtime/rest/handler/job/JobCancellationHandler.java
##########
@@ -83,7 +83,7 @@ public JobCancellationHandler(
terminationFuture = gateway.cancelJob(jobId, timeout);
break;
case STOP:
- throw new RestHandlerException("The \"stop\" command has been removed. Please use \"stop-with-savepoint\" instead.", HttpResponseStatus.PERMANENT_REDIRECT);
+ throw new RestHandlerException("The \"stop\" command has been removed. Please use \"stop\" instead.", HttpResponseStatus.PERMANENT_REDIRECT);
Review comment:
The exception message does not make much sense anymore, so it has to be updated.
----------------------------------------------------------------
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:
users@infra.apache.org
With regards,
Apache Git Services