beam-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From lukecwik <>
Subject [GitHub] beam pull request #2980: PR to run tests for #2933 using updated Dataflow wo...
Date Tue, 09 May 2017 01:04:33 GMT
GitHub user lukecwik opened a pull request:

    PR to run tests for #2933 using updated Dataflow worker image

    Be sure to do all of the following to help us incorporate your contribution
    quickly and easily:
     - [ ] Make sure the PR title is formatted like:
       `[BEAM-<Jira issue #>] Description of pull request`
     - [ ] Make sure tests pass via `mvn clean verify`.
     - [ ] Replace `<Jira issue #>` in the title with the actual Jira issue
           number, if there is one.
     - [ ] If this contribution is large, please file an Apache
           [Individual Contributor License Agreement](

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

    $ git pull finish-pr-2933

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

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #2980
commit 71ea42f5d0bdc1335bf5b9e0cb0b7fcf7db439c2
Author: Robert Bradshaw <>
Date:   2017-05-05T23:20:37Z

    Remove explicit used of nested contexts.
        find . -type f -name '*.java' | xargs sed -i '' 's/\([.]..code[(].*\),  *context.nested..[)]/\1)/'
        find . -type f -name '*.java' | xargs sed -i '' 's/\([.]..code[(].*\),  *nestedContext[)]/\1)/'
        find . -type f -name '*.java' | xargs sed -i '' 's/\([.]..code[(].*\),  *Context.NESTED[)]/\1)/'
        find . -type f -name '*.java' | xargs sed -i '' 's/\([.]..code[(].*\),  *[^ ]*.Context.NESTED[)]/\1)/'
    Added back explicit context in due to compile error.

commit 1f930b7888a923e3d8b7fbb42d26484f6a711123
Author: Robert Bradshaw <>
Date:   2017-05-05T23:36:47Z

    Remove contexts from coders where they'll never be used.

commit 98ec267b5a0e95eeb2c27b4969ffec03e0e360a8
Author: Robert Bradshaw <>
Date:   2017-05-06T00:24:02Z

    automated context removal or redirection

commit fb50c3ff26db4ecac8abaebaffc407651b294ac9
Author: Robert Bradshaw <>
Date:   2017-05-06T00:27:13Z

    get it compiling

commit 44ee9b87b603a1905e136cf7356453e3d26ed5ca
Author: Robert Bradshaw <>
Date:   2017-05-06T00:35:35Z

    Remove en/decodeOuter and default encode/decode methods.
    Now only the context-free encode() and decode() methods are abstract.

commit 7b3f2914a4195052d7e9d68e4e6c82e4ce0d8dc2
Author: Lukasz Cwik <>
Date:   2017-05-08T02:41:07Z

    fixup! Swap to use encode/decode in anonymous inner class coder and @AutoValue coder

commit 0c502352cf5d2f52f454de30dbeac2f483dbdd2d
Author: Robert Bradshaw <>
Date:   2017-05-08T17:48:40Z

    Reviewer comments + a couple of extra fixes. All compiles.

commit 0acdef7dd0417e613284f62a824c2e11b51ce999
Author: Robert Bradshaw <>
Date:   2017-05-08T18:27:40Z

    lint error

commit 52d594ec7650899f3aa3064a5c698a3378052a65
Author: Robert Bradshaw <>
Date:   2017-05-08T18:54:48Z


commit 6b81523e392af31829ae12f28a1ae0b1aba8d550
Author: Robert Bradshaw <>
Date:   2017-05-08T21:56:25Z

    Use latest dataflow worker.

commit ac626f04cd8ce5aa0862723cd208b5e7d017aaa8
Author: Robert Bradshaw <>
Date:   2017-05-08T22:04:36Z

    Explicitly mark Coder context as experimental as well as deprecated.


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

View raw message