GitHub user lukecwik opened a pull request:
https://github.com/apache/beam/pull/2980
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](https://www.apache.org/licenses/icla.pdf).
---
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/lukecwik/incubator-beam finish-pr-2933
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/beam/pull/2980.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 #2980
----
commit 71ea42f5d0bdc1335bf5b9e0cb0b7fcf7db439c2
Author: Robert Bradshaw <rober...@gmail.com>
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 CoGbkResult.java due to compile error.
commit 1f930b7888a923e3d8b7fbb42d26484f6a711123
Author: Robert Bradshaw <rober...@gmail.com>
Date: 2017-05-05T23:36:47Z
Remove contexts from coders where they'll never be used.
commit 98ec267b5a0e95eeb2c27b4969ffec03e0e360a8
Author: Robert Bradshaw <rober...@gmail.com>
Date: 2017-05-06T00:24:02Z
automated context removal or redirection
commit fb50c3ff26db4ecac8abaebaffc407651b294ac9
Author: Robert Bradshaw <rober...@gmail.com>
Date: 2017-05-06T00:27:13Z
get it compiling
commit 44ee9b87b603a1905e136cf7356453e3d26ed5ca
Author: Robert Bradshaw <rober...@gmail.com>
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 <lc...@google.com>
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 <rober...@gmail.com>
Date: 2017-05-08T17:48:40Z
Reviewer comments + a couple of extra fixes. All compiles.
commit 0acdef7dd0417e613284f62a824c2e11b51ce999
Author: Robert Bradshaw <rober...@gmail.com>
Date: 2017-05-08T18:27:40Z
lint error
commit 52d594ec7650899f3aa3064a5c698a3378052a65
Author: Robert Bradshaw <rober...@gmail.com>
Date: 2017-05-08T18:54:48Z
checkstyle
commit 6b81523e392af31829ae12f28a1ae0b1aba8d550
Author: Robert Bradshaw <rober...@gmail.com>
Date: 2017-05-08T21:56:25Z
Use latest dataflow worker.
commit ac626f04cd8ce5aa0862723cd208b5e7d017aaa8
Author: Robert Bradshaw <rober...@gmail.com>
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
