Github user tillrohrmann commented on the issue:
https://github.com/apache/flink/pull/4802
True, you're right. I guess I overlooked it. But it also includes the
`trigger` prefix which indicates that it does not take a savepoint but only
triggers it. If we want to be consistent, then we would also return a future
for the `stop` and `cancel` call on which you can wait if you want to wait for
the completion of the call. But I guess that this part will be reworked anyway
in the future when we refactor the client. Thus, I guess you can merge the PR.
---
