flink-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From florianschmidt1994 <...@git.apache.org>
Subject [GitHub] flink pull request #5807: [FLINK-8982][E2E Tests] Add test for known failure...
Date Tue, 03 Apr 2018 12:51:19 GMT
GitHub user florianschmidt1994 opened a pull request:

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

    [FLINK-8982][E2E Tests] Add test for known failure of queryable state

     ## What is the purpose of the change
    Add an end-to-end test to verify that the changes that @kl0u introduced in https://github.com/apache/flink/pull/5691
fix a known issue with concurrent access to queryable state, by verifying that access to queryable
state works as expected.
    
    ## Brief change log
    - Add flink app with queryable state the continuously updates mapstate
    - Add queryable state client that periodically queries map state
    - Add end-to-end test that runs client against app and verifies that no unexpected exceptions
occur
    - Integrate end-to-end test in testsuite
    
    ## Verifying this change
    This change added tests and can be verified as follows:
    - Run `./run-pre-commit-tests.sh`
    
    ## 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/florianschmidt1994/flink end-to-end-tests-for-queryable-state

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

    https://github.com/apache/flink/pull/5807.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 #5807
    
----
commit a94b06728c25df9a5c71a484f49cfabb36eb1460
Author: Florian Schmidt <florian.schmidt.1994@...>
Date:   2018-03-13T13:13:08Z

    [FLINK-8982][E2E Tests] Add test for known failure of queryable state

----


---

Mime
View raw message