You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@flink.apache.org by kl0u <gi...@git.apache.org> on 2016/09/21 22:27:27 UTC

[GitHub] flink pull request #2532: [FLINK-4494] Expose the TimeServiceProvider from t...

GitHub user kl0u opened a pull request:

    https://github.com/apache/flink/pull/2532

    [FLINK-4494] Expose the TimeServiceProvider from the Task to each Operator.

    This is a PR for both [FLINK-4496] and [FLINK-4494].
    R: @aljoscha 

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/kl0u/flink timeprovider_exposing

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/flink/pull/2532.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 #2532
    
----
commit e1522ace235af2a524bce96e180a7d2729c5e6a8
Author: kl0u <kk...@gmail.com>
Date:   2016-08-25T15:38:49Z

    [FLINK-4496] Refactor the TimeServiceProvider to take a Trigerable instead of a Runnable.

commit 2ebdbdafa41d04355cce4b27954c53013d0bc4ba
Author: kl0u <kk...@gmail.com>
Date:   2016-09-21T15:08:29Z

    Updating the PR.

commit 02b120f6bec36c88c9dae9f20fa4b7d74b42cdaf
Author: kl0u <kk...@gmail.com>
Date:   2016-09-20T12:45:01Z

    [FLINK-4494] Expose the TimeServiceProvider from the Task to each Operator.

----


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] flink pull request #2532: [FLINK-4494] Expose the TimeServiceProvider from t...

Posted by kl0u <gi...@git.apache.org>.

Github user kl0u closed the pull request at:

    https://github.com/apache/flink/pull/2532


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] flink issue #2532: [FLINK-4494] Expose the TimeServiceProvider from the Task...

Posted by aljoscha <gi...@git.apache.org>.

Github user aljoscha commented on the issue:

    https://github.com/apache/flink/pull/2532
  
    Good work. \U0001f44d
    
    I merged this, so could you please close the two PRs and the relevant issues.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] flink issue #2532: [FLINK-4494] Expose the TimeServiceProvider from the Task...

Posted by kl0u <gi...@git.apache.org>.

Github user kl0u commented on the issue:

    https://github.com/apache/flink/pull/2532
  
    Yes. Thanks a lot @aljoscha !



---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---