Re: Please do not merge Python PRs

2018-11-15 Thread Udi Meiri
All clear, Python tests are reporting errors correctly again. On Wed, Nov 14, 2018 at 5:57 PM Udi Meiri wrote: > https://github.com/apache/beam/pull/7048 is the rollback PR > > On Wed, Nov 14, 2018 at 5:28 PM Ahmet Altay wrote: > >> Thank you Udi. Could you send a rollback PR? >> >> I believe

Re: Please do not merge Python PRs

2018-11-14 Thread Udi Meiri
https://github.com/apache/beam/pull/7048 is the rollback PR On Wed, Nov 14, 2018 at 5:28 PM Ahmet Altay wrote: > Thank you Udi. Could you send a rollback PR? > > I believe this is https://issues.apache.org/jira/browse/BEAM-6048 > > On Wed, Nov 14, 2018 at 5:16 PM, Udi Meiri wrote: > >> It

Re: Please do not merge Python PRs

2018-11-14 Thread Ahmet Altay
Thank you Udi. Could you send a rollback PR? I believe this is https://issues.apache.org/jira/browse/BEAM-6048 On Wed, Nov 14, 2018 at 5:16 PM, Udi Meiri wrote: > It seems that Gradle is not getting the correct exit status from test runs. > Possible culprit:

Re: Please do not merge Python PRs

2018-11-14 Thread Udi Meiri
Recreated locally: https://gradle.com/s/psqgcywnc3h2m On Wed, Nov 14, 2018 at 5:16 PM Udi Meiri wrote: > It seems that Gradle is not getting the correct exit status from test runs. > Possible culprit: https://github.com/apache/beam/pull/6903 > smime.p7s Description: S/MIME Cryptographic

Please do not merge Python PRs

2018-11-14 Thread Udi Meiri
It seems that Gradle is not getting the correct exit status from test runs. Possible culprit: https://github.com/apache/beam/pull/6903 smime.p7s Description: S/MIME Cryptographic Signature