[
https://issues.apache.org/jira/browse/FLINK-8632?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16359678#comment-16359678
]
ASF GitHub Bot commented on FLINK-8632:
---------------------------------------
GitHub user tillrohrmann opened a pull request:
https://github.com/apache/flink/pull/5451
[FLINK-8632] [flip6] Introduce generalized asynchronous operation handlers
## What is the purpose of the change
The asynchronous operation handlers are the generalization of the
SavepointHandlers.
They consist of a Trigger- and a StatusHandler. The TriggerHandler is used
to trigger
an asynchronous operation. The handler stores the operation future and
returns a
trigger id. The trigger id can be used to query the status of the operation
via the
StatusHandler. Once the operation has completed, the StatusHandler will
return the
result.
This PR is based on #5450.
## Brief change log
- Introduced `AbstractAsynchronousOperationHandlers` derived from
`SavepointHandlers`
- Reworked `SavepointHandlers` according to
`AbstractAsynchronousOperationHandlers` interface
## Verifying this change
- Added `AbstractAsynchronousOperationHandlersTest`
- Also tested by `SavepointHandlersTest`
## Does this pull request potentially affect one of the following parts:
- Dependencies (does it add or upgrade a dependency): (no)
- The public API, i.e., is any changed class annotated with
`@Public(Evolving)`: (no)
- The serializers: (no)
- The runtime per-record code paths (performance sensitive): (no)
- Anything that affects deployment or recovery: JobManager (and its
components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
- The S3 file system connector: (no)
## Documentation
- Does this pull request introduce a new feature? (no)
- If yes, how is the feature documented? (not applicable)
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/tillrohrmann/flink
asynchronousOperationHandlers
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/flink/pull/5451.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #5451
----
commit b0efaa09c3b39e31ba130c668abe39af40473fb4
Author: Till Rohrmann <trohrmann@...>
Date: 2018-02-03T16:41:51Z
[FLINK-8631] [rest] Add support for generic types to the RestClient
This commit allows the Restclient to receive generic response types. In
order
to do this, the MessageHeaders contain now information about the generic
type parameters of the response type.
commit e26cf11281b48bd04372f12363397c9267e4413a
Author: Till Rohrmann <trohrmann@...>
Date: 2018-02-02T10:23:00Z
[FLINK-8632] [flip6] Introduce generalized asynchronous operation handlers
The asynchronous operation handlers are the generalization of the
SavepointHandlers.
They consist of a Trigger- and a StatusHandler. The TriggerHandler is used
to trigger
an asynchronous operation. The handler stores the operation future and
returns a
trigger id. The trigger id can be used to query the status of the operation
via the
StatusHandler. Once the operation has completed, the StatusHandler will
return the
result.
----
> Generalize SavepointHandlers to be used for other asynchronous operations
> -------------------------------------------------------------------------
>
> Key: FLINK-8632
> URL: https://issues.apache.org/jira/browse/FLINK-8632
> Project: Flink
> Issue Type: Improvement
> Components: REST
> Affects Versions: 1.5.0
> Reporter: Till Rohrmann
> Assignee: Till Rohrmann
> Priority: Major
> Labels: flip-6
> Fix For: 1.5.0
>
>
> We should generalize the {{SavepointHandlers}} to be usable for other
> asynchronous operations as well. The basic idea is that one has a trigger
> handler which triggers an asynchronous operation. This operation returns a
> future which is completed once the operation is done. The trigger handler
> returns a trigger id which can be used to check the status of the operation
> by querying the status handler.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
