[GitHub] jkff commented on issue #4172: [BEAM-3243] support multiple anonymous classes from the same enclosing class in a pipeline

2017-11-26 Thread GitBox
jkff commented on issue #4172: [BEAM-3243] support multiple anonymous classes from the same enclosing class in a pipeline URL: https://github.com/apache/beam/pull/4172#issuecomment-347018533 Using multiple anonymous DoFn's with the same enclosing class within the same composite transform i

[GitHub] rmannibucau commented on issue #4172: [BEAM-3243] support multiple anonymous classes from the same enclosing class in a pipeline

2017-11-26 Thread GitBox
rmannibucau commented on issue #4172: [BEAM-3243] support multiple anonymous classes from the same enclosing class in a pipeline URL: https://github.com/apache/beam/pull/4172#issuecomment-347019445 I understand where it comes from but it is very boring when writing tests where it is not un

[GitHub] jkff commented on issue #4172: [BEAM-3243] support multiple anonymous classes from the same enclosing class in a pipeline

2017-11-26 Thread GitBox
jkff commented on issue #4172: [BEAM-3243] support multiple anonymous classes from the same enclosing class in a pipeline URL: https://github.com/apache/beam/pull/4172#issuecomment-347029760 Yeah, I would prefer an improved error message. I suppose you mean the message that says `Transform

[GitHub] rmannibucau commented on issue #4172: [BEAM-3243] support multiple anonymous classes from the same enclosing class in a pipeline

2017-11-26 Thread GitBox
rmannibucau commented on issue #4172: [BEAM-3243] support multiple anonymous classes from the same enclosing class in a pipeline URL: https://github.com/apache/beam/pull/4172#issuecomment-347035725 Something like Transform Foo$1 conflicts with Foo$2 in pipeline defined in MyTest

Re: [VOTE] Release 2.2.0, release candidate #4

2017-11-26 Thread Reuven Lax
FYI I'm still working on finalizing the release. The issue is that the Beam documentation underwent a large refactor while the release was pending, so the website PR (pr/337) conflicts. I think it's safer to give up on resolving these conflicts, so I'm simply going to regenerate this PR from scratc

[GitHub] tweise commented on issue #4074: [BEAM-3130] View.asMap() causes a ClassCastException in Apex runner

2017-11-26 Thread GitBox
tweise commented on issue #4074: [BEAM-3130] View.asMap() causes a ClassCastException in Apex runner URL: https://github.com/apache/beam/pull/4074#issuecomment-347068452 @jkff tests are added This is an automated message from

[GitHub] luke-zhu opened a new pull request #4176: [BEAM-3143] Type Inference Compatibility with Python 3

2017-11-26 Thread GitBox
luke-zhu opened a new pull request #4176: [BEAM-3143] Type Inference Compatibility with Python 3 URL: https://github.com/apache/beam/pull/4176 Follow this checklist to help us incorporate your contribution quickly and easily: - [ ] Make sure there is a [JIRA issue](https://issues.

[GitHub] chamikaramj commented on a change in pull request #4136: [BEAM-3184] Added ProxyInfoFromEnvironmentVar() & GetNewHttp() methods for GCS

2017-11-26 Thread GitBox
chamikaramj commented on a change in pull request #4136: [BEAM-3184] Added ProxyInfoFromEnvironmentVar() & GetNewHttp() methods for GCS URL: https://github.com/apache/beam/pull/4136#discussion_r153058028 ## File path: sdks/python/apache_beam/io/gcp/gcsio.py ## @@ -87,6 +87

[GitHub] chamikaramj commented on a change in pull request #4136: [BEAM-3184] Added ProxyInfoFromEnvironmentVar() & GetNewHttp() methods for GCS

2017-11-26 Thread GitBox
chamikaramj commented on a change in pull request #4136: [BEAM-3184] Added ProxyInfoFromEnvironmentVar() & GetNewHttp() methods for GCS URL: https://github.com/apache/beam/pull/4136#discussion_r153057996 ## File path: sdks/python/apache_beam/io/gcp/gcsio.py ## @@ -87,6 +87

[GitHub] chamikaramj commented on a change in pull request #4136: [BEAM-3184] Added ProxyInfoFromEnvironmentVar() & GetNewHttp() methods for GCS

2017-11-26 Thread GitBox
chamikaramj commented on a change in pull request #4136: [BEAM-3184] Added ProxyInfoFromEnvironmentVar() & GetNewHttp() methods for GCS URL: https://github.com/apache/beam/pull/4136#discussion_r153058033 ## File path: sdks/python/apache_beam/io/gcp/gcsio.py ## @@ -87,6 +87

[GitHub] chamikaramj commented on a change in pull request #4136: [BEAM-3184] Added ProxyInfoFromEnvironmentVar() & GetNewHttp() methods for GCS

2017-11-26 Thread GitBox
chamikaramj commented on a change in pull request #4136: [BEAM-3184] Added ProxyInfoFromEnvironmentVar() & GetNewHttp() methods for GCS URL: https://github.com/apache/beam/pull/4136#discussion_r153058019 ## File path: sdks/python/apache_beam/io/gcp/gcsio.py ## @@ -87,6 +87

[GitHub] chamikaramj commented on a change in pull request #4136: [BEAM-3184] Added ProxyInfoFromEnvironmentVar() & GetNewHttp() methods for GCS

2017-11-26 Thread GitBox
chamikaramj commented on a change in pull request #4136: [BEAM-3184] Added ProxyInfoFromEnvironmentVar() & GetNewHttp() methods for GCS URL: https://github.com/apache/beam/pull/4136#discussion_r153057981 ## File path: sdks/python/apache_beam/io/gcp/gcsio.py ## @@ -87,6 +87

[GitHub] chamikaramj commented on a change in pull request #4136: [BEAM-3184] Added ProxyInfoFromEnvironmentVar() & GetNewHttp() methods for GCS

2017-11-26 Thread GitBox
chamikaramj commented on a change in pull request #4136: [BEAM-3184] Added ProxyInfoFromEnvironmentVar() & GetNewHttp() methods for GCS URL: https://github.com/apache/beam/pull/4136#discussion_r153058069 ## File path: sdks/python/apache_beam/io/gcp/gcsio.py ## @@ -87,6 +87

[GitHub] chamikaramj commented on a change in pull request #4136: [BEAM-3184] Added ProxyInfoFromEnvironmentVar() & GetNewHttp() methods for GCS

2017-11-26 Thread GitBox
chamikaramj commented on a change in pull request #4136: [BEAM-3184] Added ProxyInfoFromEnvironmentVar() & GetNewHttp() methods for GCS URL: https://github.com/apache/beam/pull/4136#discussion_r153058006 ## File path: sdks/python/apache_beam/io/gcp/gcsio.py ## @@ -87,6 +87

[GitHub] chamikaramj commented on a change in pull request #4136: [BEAM-3184] Added ProxyInfoFromEnvironmentVar() & GetNewHttp() methods for GCS

2017-11-26 Thread GitBox
chamikaramj commented on a change in pull request #4136: [BEAM-3184] Added ProxyInfoFromEnvironmentVar() & GetNewHttp() methods for GCS URL: https://github.com/apache/beam/pull/4136#discussion_r153057990 ## File path: sdks/python/apache_beam/io/gcp/gcsio.py ## @@ -87,6 +87

[GitHub] jbonofre commented on issue #3808: [BEAM-1920] Add a Spark 2.x support in the Spark runner

2017-11-26 Thread GitBox
jbonofre commented on issue #3808: [BEAM-1920] Add a Spark 2.x support in the Spark runner URL: https://github.com/apache/beam/pull/3808#issuecomment-347089270 Following the vote on the mailing lists, I updated the PR with Spark 2.x update only (no support of Spark 1 anymore). ---

GitBox down

2017-11-26 Thread Jean-Baptiste Onofré
Hi, it seems GitBox is down (timing out). I don't see anything on status.apache.org, I will ping INFRA. Regards JB -- Jean-Baptiste Onofré jbono...@apache.org http://blog.nanthrax.net Talend - http://www.talend.com

Re: [DISCUSS] Drop Spark 1.x support to focus on Spark 2.x

2017-11-26 Thread Jean-Baptiste Onofré
Hi all, Quick update about the Spark 2.x runner: I updated the PR with Spark 2.x update only: https://github.com/apache/beam/pull/3808 I will rebase and do new tests as soon as gitbox will be back. Don't hesitate to take a look and review. Thanks ! Regards JB On 11/21/2017 08:32 AM, Jean-B