[GitHub] flink pull request: [FLINK-2098] Improvements on checkpoint-aligne...

2015-06-01 Thread StephanEwen
GitHub user StephanEwen opened a pull request:

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

[FLINK-2098] Improvements on checkpoint-aligned sources

Based on #742 , include multiple cleanups and fixes on top.

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

$ git pull https://github.com/StephanEwen/incubator-flink stream_sources

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

https://github.com/apache/flink/pull/755.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 #755


commit 152f4af6966e8a2e770f20a1f516d1de29c5b13e
Author: twalthr 
Date:   2015-05-27T13:32:11Z

[hotfix] Remove execute() after print() in Table API examples

This closes #735

commit d82942a396777ea6debbf55a91916d4d5c3ecdaa
Author: Aljoscha Krettek 
Date:   2015-05-28T08:24:37Z

[FLINK-2098] Ensure checkpoints and element emission are in order

Before, it could happen that a streaming source would keep emitting
elements while a checkpoint is being performed. This can lead to
inconsistencies in the checkpoints with other operators.

This also adds a test that checks whether only one checkpoint is
executed at a time and the serial behaviour of checkpointing and
emission.

This changes the SourceFunction interface to have run()/cancel() methods
where the run() method takes a lock object on which it needs to
synchronize updates to state and emission of elements.

commit dad6a0092489fe7a9ef4508dc006d5a6cc42a2ba
Author: Stephan Ewen 
Date:   2015-06-02T01:31:43Z

[FLINK-2098] Improvements on checkpoint-aligned sources




---
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.
---


[GitHub] flink pull request: [FLINK-2098] Improvements on checkpoint-aligne...

2015-06-03 Thread aljoscha
Github user aljoscha commented on the pull request:

https://github.com/apache/flink/pull/755#issuecomment-108218506
  
@StephanEwen I took the changes, added them on top of my PR and added some 
more refinements.


---
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.
---


[GitHub] flink pull request: [FLINK-2098] Improvements on checkpoint-aligne...

2015-06-05 Thread StephanEwen
Github user StephanEwen closed the pull request at:

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


---
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.
---


[GitHub] flink pull request: [FLINK-2098] Improvements on checkpoint-aligne...

2015-06-05 Thread StephanEwen
Github user StephanEwen commented on the pull request:

https://github.com/apache/flink/pull/755#issuecomment-109442172
  
merged as part of #742


---
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.
---