[GitHub] flink pull request #2072: [FLINK-3948] Protect RocksDB cleanup by cleanup lo...

[aljoscha]

Github user aljoscha closed the pull request at:

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


---
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 #2072: [FLINK-3948] Protect RocksDB cleanup by cleanup lo...

[aljoscha]

GitHub user aljoscha opened a pull request:

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

[FLINK-3948] Protect RocksDB cleanup by cleanup lock

Before, it could happen that an asynchronous checkpoint was going on
when trying to do cleanup. Now we protect cleanup and asynchronous
checkpointing by a lock.

This was what caused `EventTimeWindowCheckpointingITCase` to fail. I now 
ran it more than a 100 times on travis and haven't observed a build failure 
related to this.

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

$ git pull https://github.com/aljoscha/flink rocksdb/fix-core-dump

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

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


commit c8456b45c47e67cc316d5bb979de36a6225eebd4
Author: Aljoscha Krettek 
Date:   2016-06-04T05:59:48Z

Revert "[FLINK-3960] ignore EventTimeWindowCheckpointingITCase for now"

This reverts commit 98a939552e12fc699ff39111bbe877e112460ceb.

commit 13c8593ec9074aa086caf4329b21e331a1c54d58
Author: Aljoscha Krettek 
Date:   2016-05-20T20:37:14Z

[FLINK-3948] Protect RocksDB cleanup by cleanup lock

Before, it could happen that an asynchronous checkpoint was going on
when trying to do cleanup. Now we protect cleanup and asynchronous
checkpointing by a lock.




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