Skip to content

Conversation

@zohar-mizrahi
Copy link

Thanks for contributing to Apache Flink. Before you open your pull request, please take the following check list into consideration.
If your changes take all of the items into account, feel free to open your pull request. For more information and/or questions please refer to the How To Contribute guide.
In addition to going through the list, please provide a meaningful description of your changes.

  • General

    • The pull request references the related JIRA issue ("[FLINK-XXX] Jira title text")
    • The pull request addresses only one issue
    • Each commit in the PR has a meaningful commit message (including the JIRA id)
  • Documentation

    • Documentation has been added for new functionality
    • Old documentation affected by the pull request has been updated
    • JavaDoc for public methods has been added
  • Tests & Build

    • Functionality added by the pull request is covered by tests
    • mvn clean verify has been executed successfully locally or a Travis build has passed

rmetzger and others added 30 commits January 3, 2017 10:06
…ation of migration package classes, override resolveClass(...) instead

This closes apache#3050
- Added info about checkpointing changing the default restart
strategy in places where it was missing: the config page and the
section about the fixed-delay strategy
- Replaced no-restart with "no restart" so people don't think we're
 referring to a config value
- Replaced invalid <it> html tag with <code>
- Fixed bad link to restart strategies page from state.md
This test ensures that we set a low-level processing time timer in case
we have processing-time timers set.
StephanEwen and others added 26 commits January 25, 2017 12:08
Introduces a null check to deal with state futures which have a null value.

This closes apache#3213.
…tOperators

This PR addresses the problem by changing the Emitter's behaviour to first output the
element before removing it from the StreamElementQueue. That way the close method waits
until also the Emitter has outputted the last completed element. Additionally, the
stopResources method now frees the checkpoint lock in order to let the emitter thread
react to the interrupt signal.

This closes apache#3210.
Previously, deleting in HA mode was only tested with a local file system.
This verifies that the delete still works on HDFS.

This closes apache#3222.
The test was flakey, because the expected behaviour is actually
not possible with the current state of queryable state, which
allows uncommitted reads.
When calling PendingCheckpoint.dispose, the state contained of a pending checkpoint
is discarded by an asynchronous task. Since this task accesses the taskStates field
we must not clear it in PendingCheckpoint.dispose. Instead we will clear it once
all state objects have been discarded from within the asynchronous task.

This closes apache#3221.
We have to change the instance path to not include too many nested
directories, otherwise the Keyed backend cannot properly clean up the
whole directory hierarchy.
… operation

This PR synchronizes asynchronous checkpointing and close operations of a StreamTask.
The synchronization prevents that an acknowledged checkpoint gets discarded and that
a discarded checkpoint gets acknowledged. It achieves this by introducing an atomic
state variable which guards against late close and acknowledge operations.

This closes apache#3227.
…d inherit from StreamingMultipleProgramsTestBase
@zentol
Copy link
Contributor

zentol commented May 8, 2017

Please close this pull request as it is subsumed by #3838.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.