[GitHub] [beam-wheels] aaltay merged pull request #18: Deprecate repository, not used in release process anymore

2020-07-30 Thread GitBox
aaltay merged pull request #18: URL: https://github.com/apache/beam-wheels/pull/18 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to t

[GitHub] [beam-wheels] aaltay commented on pull request #18: Deprecate repository, not used in release process anymore

2020-07-29 Thread GitBox
aaltay commented on pull request #18: URL: https://github.com/apache/beam-wheels/pull/18#issuecomment-665798992 Nice. LGTM. @tvalentyn - Could you verify that 2.23 release did not use this repo? This is an automated me

[GitHub] [beam-wheels] TobKed commented on pull request #18: Deprecate repository (depends on https://github.com/apache/beam/pull/12150)

2020-07-29 Thread GitBox
TobKed commented on pull request #18: URL: https://github.com/apache/beam-wheels/pull/18#issuecomment-665693872 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and u

[GitHub] [beam-wheels] TobKed opened a new pull request #18: Deprecate repository (depends on https://github.com/apache/beam/pull/12150)

2020-07-01 Thread GitBox
TobKed opened a new pull request #18: URL: https://github.com/apache/beam-wheels/pull/18 When GitHub Actions used to build Python Source Distribution and Wheels will be used in release process this repository may be deprecated. https://github.com/apache/beam/pull/12150

[GitHub] [beam-wheels] brucearctor opened a new pull request #17: Update README.md

2020-05-26 Thread GitBox
brucearctor opened a new pull request #17: URL: https://github.com/apache/beam-wheels/pull/17 @aaltay @pabloem ? This is an automated message from the Apache Git Service. To respond to the message, please log on to G

[GitHub] [beam-wheels] ibzib commented on a change in pull request #17: Update README.md

2020-05-26 Thread GitBox
ibzib commented on a change in pull request #17: URL: https://github.com/apache/beam-wheels/pull/17#discussion_r430645550 ## File path: README.md ## @@ -32,9 +32,11 @@ There are 2 major parts in this repository. * [travis](https://travis-ci.com/) configuration files, setups en

[GitHub] [beam-wheels] ibzib merged pull request #17: Update README.md

2020-05-26 Thread GitBox
ibzib merged pull request #17: URL: https://github.com/apache/beam-wheels/pull/17 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to th

[GitHub] [beam-wheels] aaltay commented on issue #15: update travis config based on https://config.travis-ci.com/explore

2020-04-02 Thread GitBox
aaltay commented on issue #15: update travis config based on https://config.travis-ci.com/explore URL: https://github.com/apache/beam-wheels/pull/15#issuecomment-608175122 > This change has fixed the "does not trigger build" problem. Thank you! --

[GitHub] [beam-wheels] amaliujia merged pull request #16: Fix travis config

2020-04-02 Thread GitBox
amaliujia merged pull request #16: Fix travis config URL: https://github.com/apache/beam-wheels/pull/16 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [beam-wheels] amaliujia opened a new pull request #16: Fix travis config

2020-04-02 Thread GitBox
amaliujia opened a new pull request #16: Fix travis config URL: https://github.com/apache/beam-wheels/pull/16 R: @boyuanzz This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [beam-wheels] amaliujia commented on issue #15: update travis config based on https://config.travis-ci.com/explore

2020-04-02 Thread GitBox
amaliujia commented on issue #15: update travis config based on https://config.travis-ci.com/explore URL: https://github.com/apache/beam-wheels/pull/15#issuecomment-608120384 This change has fixed the "does not trigger build" problem. ---

[GitHub] [beam-wheels] amaliujia merged pull request #15: update travis config based on https://config.travis-ci.com/explore

2020-04-02 Thread GitBox
amaliujia merged pull request #15: update travis config based on https://config.travis-ci.com/explore URL: https://github.com/apache/beam-wheels/pull/15 This is an automated message from the Apache Git Service. To respond to

[GitHub] [beam-wheels] amaliujia commented on a change in pull request #15: update travis config based on https://config.travis-ci.com/explore

2020-04-02 Thread GitBox
amaliujia commented on a change in pull request #15: update travis config based on https://config.travis-ci.com/explore URL: https://github.com/apache/beam-wheels/pull/15#discussion_r402624271 ## File path: .travis.yml ## @@ -96,9 +95,9 @@ deploy: access_key_id: ${ACCES

[GitHub] [beam-wheels] aaltay commented on a change in pull request #15: update travis config based on https://config.travis-ci.com/explore

2020-04-02 Thread GitBox
aaltay commented on a change in pull request #15: update travis config based on https://config.travis-ci.com/explore URL: https://github.com/apache/beam-wheels/pull/15#discussion_r402623417 ## File path: .travis.yml ## @@ -16,7 +15,7 @@ env: - BUILD_DEPENDS="Cython"

[GitHub] [beam-wheels] aaltay commented on a change in pull request #15: update travis config based on https://config.travis-ci.com/explore

2020-04-02 Thread GitBox
aaltay commented on a change in pull request #15: update travis config based on https://config.travis-ci.com/explore URL: https://github.com/apache/beam-wheels/pull/15#discussion_r402622997 ## File path: .travis.yml ## @@ -96,9 +95,9 @@ deploy: access_key_id: ${ACCESS_K

[GitHub] [beam-wheels] amaliujia commented on a change in pull request #15: update travis config based on https://config.travis-ci.com/explore

2020-04-02 Thread GitBox
amaliujia commented on a change in pull request #15: update travis config based on https://config.travis-ci.com/explore URL: https://github.com/apache/beam-wheels/pull/15#discussion_r402622210 ## File path: .travis.yml ## @@ -16,7 +15,7 @@ env: - BUILD_DEPENDS="Cyth

[GitHub] [beam-wheels] amaliujia edited a comment on issue #15: update travis config based on https://config.travis-ci.com/explore

2020-04-02 Thread GitBox
amaliujia edited a comment on issue #15: update travis config based on https://config.travis-ci.com/explore URL: https://github.com/apache/beam-wheels/pull/15#issuecomment-608111440 @aaltay @boyuanzz This is an automated

[GitHub] [beam-wheels] amaliujia commented on a change in pull request #15: update travis config based on https://config.travis-ci.com/explore

2020-04-02 Thread GitBox
amaliujia commented on a change in pull request #15: update travis config based on https://config.travis-ci.com/explore URL: https://github.com/apache/beam-wheels/pull/15#discussion_r402620104 ## File path: .travis.yml ## @@ -16,7 +15,7 @@ env: - BUILD_DEPENDS="Cyth

[GitHub] [beam-wheels] amaliujia commented on issue #15: update travis config based on https://config.travis-ci.com/explore

2020-04-02 Thread GitBox
amaliujia commented on issue #15: update travis config based on https://config.travis-ci.com/explore URL: https://github.com/apache/beam-wheels/pull/15#issuecomment-608111440 @aaltay This is an automated message from the

[GitHub] [beam-wheels] amaliujia opened a new pull request #15: update travis config based on https://config.travis-ci.com/explore

2020-04-02 Thread GitBox
amaliujia opened a new pull request #15: update travis config based on https://config.travis-ci.com/explore URL: https://github.com/apache/beam-wheels/pull/15 Travis-ci now has an online tool to validate travis config: https://config.travis-ci.com/explore. I updated travis config b

[GitHub] [beam] kamilwu commented on a change in pull request #11194: [BEAM-9507] Fix python dependency check task

2020-03-24 Thread GitBox
kamilwu commented on a change in pull request #11194: [BEAM-9507] Fix python dependency check task URL: https://github.com/apache/beam/pull/11194#discussion_r397019242 ## File path: sdks/python/build.gradle ## @@ -89,6 +89,7 @@ task depSnapshot { task dependencyUpdates

[GitHub] [beam] kamilwu commented on a change in pull request #11194: [BEAM-9507] Fix python dependency check task

2020-03-24 Thread GitBox
kamilwu commented on a change in pull request #11194: [BEAM-9507] Fix python dependency check task URL: https://github.com/apache/beam/pull/11194#discussion_r397019242 ## File path: sdks/python/build.gradle ## @@ -89,6 +89,7 @@ task depSnapshot { task dependencyUpdates

[GitHub] [beam] ihji opened a new pull request #11205: [BEAM-9578] Enumerating artifacts is too expensive in Java

2020-03-24 Thread GitBox
ihji opened a new pull request #11205: [BEAM-9578] Enumerating artifacts is too expensive in Java URL: https://github.com/apache/beam/pull/11205 Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and e

[GitHub] [beam] kamilwu commented on issue #11194: [BEAM-9507] Fix python dependency check task

2020-03-24 Thread GitBox
kamilwu commented on issue #11194: [BEAM-9507] Fix python dependency check task URL: https://github.com/apache/beam/pull/11194#issuecomment-603115281 Retest this please This is an automated message from the Apache Git Service.

[GitHub] [beam] alexvanboxel commented on a change in pull request #10529: [BEAM-9044] Protobuf options to Schema options

2020-03-24 Thread GitBox
alexvanboxel commented on a change in pull request #10529: [BEAM-9044] Protobuf options to Schema options URL: https://github.com/apache/beam/pull/10529#discussion_r396977813 ## File path: sdks/java/extensions/protobuf/src/main/java/org/apache/beam/sdk/extensions/protobuf/ProtoSche

[GitHub] [beam] stale[bot] commented on issue #10038: Simplify Python test process

2020-03-23 Thread GitBox
stale[bot] commented on issue #10038: Simplify Python test process URL: https://github.com/apache/beam/pull/10038#issuecomment-603041838 This pull request has been closed due to lack of activity. If you think that is incorrect, or the pull request requires review, you can revive the PR at a

[GitHub] [beam] stale[bot] closed pull request #10038: Simplify Python test process

2020-03-23 Thread GitBox
stale[bot] closed pull request #10038: Simplify Python test process URL: https://github.com/apache/beam/pull/10038 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub a

[GitHub] [beam] pabloem commented on issue #11202: asdletmedah

2020-03-23 Thread GitBox
pabloem commented on issue #11202: asdletmedah URL: https://github.com/apache/beam/pull/11202#issuecomment-603028147 Run Portable_Python PreCommit This is an automated message from the Apache Git Service. To respond to the mes

[GitHub] [beam] youngoli commented on a change in pull request #11197: [BEAM-8292] Portable Reshuffle for Go SDK

2020-03-23 Thread GitBox
youngoli commented on a change in pull request #11197: [BEAM-8292] Portable Reshuffle for Go SDK URL: https://github.com/apache/beam/pull/11197#discussion_r396853348 ## File path: sdks/go/pkg/beam/core/runtime/exec/reshuffle.go ## @@ -0,0 +1,170 @@ +// Licensed to the Apac

[GitHub] [beam] youngoli commented on a change in pull request #11197: [BEAM-8292] Portable Reshuffle for Go SDK

2020-03-23 Thread GitBox
youngoli commented on a change in pull request #11197: [BEAM-8292] Portable Reshuffle for Go SDK URL: https://github.com/apache/beam/pull/11197#discussion_r396887810 ## File path: sdks/go/pkg/beam/gbk.go ## @@ -95,3 +95,52 @@ func TryCoGroupByKey(s Scope, cols ...PCollecti

[GitHub] [beam] youngoli commented on a change in pull request #11197: [BEAM-8292] Portable Reshuffle for Go SDK

2020-03-23 Thread GitBox
youngoli commented on a change in pull request #11197: [BEAM-8292] Portable Reshuffle for Go SDK URL: https://github.com/apache/beam/pull/11197#discussion_r396823199 ## File path: sdks/go/pkg/beam/core/runtime/exec/coder.go ## @@ -259,59 +298,82 @@ type customDecoder struc

[GitHub] [beam] youngoli commented on a change in pull request #11197: [BEAM-8292] Portable Reshuffle for Go SDK

2020-03-23 Thread GitBox
youngoli commented on a change in pull request #11197: [BEAM-8292] Portable Reshuffle for Go SDK URL: https://github.com/apache/beam/pull/11197#discussion_r396886566 ## File path: sdks/go/pkg/beam/core/runtime/graphx/translate.go ## @@ -473,6 +479,164 @@ func (m *marshalle

[GitHub] [beam] youngoli commented on a change in pull request #11197: [BEAM-8292] Portable Reshuffle for Go SDK

2020-03-23 Thread GitBox
youngoli commented on a change in pull request #11197: [BEAM-8292] Portable Reshuffle for Go SDK URL: https://github.com/apache/beam/pull/11197#discussion_r396887966 ## File path: sdks/go/pkg/beam/gbk.go ## @@ -95,3 +95,52 @@ func TryCoGroupByKey(s Scope, cols ...PCollecti

[GitHub] [beam] youngoli commented on a change in pull request #11197: [BEAM-8292] Portable Reshuffle for Go SDK

2020-03-23 Thread GitBox
youngoli commented on a change in pull request #11197: [BEAM-8292] Portable Reshuffle for Go SDK URL: https://github.com/apache/beam/pull/11197#discussion_r396855047 ## File path: sdks/go/pkg/beam/core/runtime/exec/reshuffle.go ## @@ -0,0 +1,170 @@ +// Licensed to the Apac

[GitHub] [beam] youngoli commented on a change in pull request #11197: [BEAM-8292] Portable Reshuffle for Go SDK

2020-03-23 Thread GitBox
youngoli commented on a change in pull request #11197: [BEAM-8292] Portable Reshuffle for Go SDK URL: https://github.com/apache/beam/pull/11197#discussion_r396856077 ## File path: sdks/go/pkg/beam/core/runtime/exec/reshuffle.go ## @@ -0,0 +1,170 @@ +// Licensed to the Apac

[GitHub] [beam] chrisgorgo commented on issue #11204: [BEAM-9579] Fix numpy logic operators

2020-03-23 Thread GitBox
chrisgorgo commented on issue #11204: [BEAM-9579] Fix numpy logic operators URL: https://github.com/apache/beam/pull/11204#issuecomment-603002452 R: @aaltay @charlesccychen This is an automated message from the Apache Git Serv

[GitHub] [beam] chrisgorgo opened a new pull request #11204: [BEAM-9579] Fix numpy logic operators

2020-03-23 Thread GitBox
chrisgorgo opened a new pull request #11204: [BEAM-9579] Fix numpy logic operators URL: https://github.com/apache/beam/pull/11204 Replacing deprecated '-' logical operators. Thank you for your contribution! Follow this checklist to help us incorporate

[GitHub] [beam] robertwb opened a new pull request #11203: [BEAM-9577] Define and implement dependency-aware artifact staging service.

2020-03-23 Thread GitBox
robertwb opened a new pull request #11203: [BEAM-9577] Define and implement dependency-aware artifact staging service. URL: https://github.com/apache/beam/pull/11203 This is not yet used anywhere (and even the legacy service is not yet used in Dataflow) but it is good to define what we wan

[GitHub] [beam] reuvenlax commented on issue #11074: Store logical type values in Row instead of base values

2020-03-23 Thread GitBox
reuvenlax commented on issue #11074: Store logical type values in Row instead of base values URL: https://github.com/apache/beam/pull/11074#issuecomment-602935452 After 8 runs, the only Java Precommit failures have been random flakes (e.g. in Flink tests). ---

[GitHub] [beam] reuvenlax merged pull request #11074: Store logical type values in Row instead of base values

2020-03-23 Thread GitBox
reuvenlax merged pull request #11074: Store logical type values in Row instead of base values URL: https://github.com/apache/beam/pull/11074 This is an automated message from the Apache Git Service. To respond to the message

[GitHub] [beam] TheNeuralBit commented on issue #10055: [BEAM-8603] Add Python SqlTransform example script

2020-03-23 Thread GitBox
TheNeuralBit commented on issue #10055: [BEAM-8603] Add Python SqlTransform example script URL: https://github.com/apache/beam/pull/10055#issuecomment-602931763 Run XVR_Flink PostCommit This is an automated message from the A

[GitHub] [beam] TheNeuralBit commented on issue #10055: [BEAM-8603] Add Python SqlTransform example script

2020-03-23 Thread GitBox
TheNeuralBit commented on issue #10055: [BEAM-8603] Add Python SqlTransform example script URL: https://github.com/apache/beam/pull/10055#issuecomment-602931816 Run XVR_Spark PostCommit This is an automated message from the A

[GitHub] [beam] stale[bot] closed pull request #8801: Make temp_location an attribute of the StandardOptions class

2020-03-23 Thread GitBox
stale[bot] closed pull request #8801: Make temp_location an attribute of the StandardOptions class URL: https://github.com/apache/beam/pull/8801 This is an automated message from the Apache Git Service. To respond to the mes

[GitHub] [beam] stale[bot] commented on issue #8884: Add a PCollectionCache ABC and several file-based implementations

2020-03-23 Thread GitBox
stale[bot] commented on issue #8884: Add a PCollectionCache ABC and several file-based implementations URL: https://github.com/apache/beam/pull/8884#issuecomment-602929552 This pull request has been closed due to lack of activity. If you think that is incorrect, or the pull request require

[GitHub] [beam] stale[bot] closed pull request #8884: Add a PCollectionCache ABC and several file-based implementations

2020-03-23 Thread GitBox
stale[bot] closed pull request #8884: Add a PCollectionCache ABC and several file-based implementations URL: https://github.com/apache/beam/pull/8884 This is an automated message from the Apache Git Service. To respond to th

[GitHub] [beam] stale[bot] commented on issue #8801: Make temp_location an attribute of the StandardOptions class

2020-03-23 Thread GitBox
stale[bot] commented on issue #8801: Make temp_location an attribute of the StandardOptions class URL: https://github.com/apache/beam/pull/8801#issuecomment-602929558 This pull request has been closed due to lack of activity. If you think that is incorrect, or the pull request requires rev

[GitHub] [beam] pabloem opened a new pull request #11202: asdletmedah

2020-03-23 Thread GitBox
pabloem opened a new pull request #11202: asdletmedah URL: https://github.com/apache/beam/pull/11202 **Please** add a meaningful description for your change here Thank you for your contribution! Follow this checklist to help us incorporate your contrib

[GitHub] [beam] pabloem commented on a change in pull request #11163: [BEAM-9548] Add better error handling to the TestStreamServiceController

2020-03-23 Thread GitBox
pabloem commented on a change in pull request #11163: [BEAM-9548] Add better error handling to the TestStreamServiceController URL: https://github.com/apache/beam/pull/11163#discussion_r396767924 ## File path: sdks/python/apache_beam/runners/interactive/caching/streaming_cache.py

[GitHub] [beam] pabloem commented on a change in pull request #11163: [BEAM-9548] Add better error handling to the TestStreamServiceController

2020-03-23 Thread GitBox
pabloem commented on a change in pull request #11163: [BEAM-9548] Add better error handling to the TestStreamServiceController URL: https://github.com/apache/beam/pull/11163#discussion_r396810340 ## File path: sdks/python/apache_beam/runners/interactive/interactive_runner.py ##

[GitHub] [beam] pabloem commented on a change in pull request #11163: [BEAM-9548] Add better error handling to the TestStreamServiceController

2020-03-23 Thread GitBox
pabloem commented on a change in pull request #11163: [BEAM-9548] Add better error handling to the TestStreamServiceController URL: https://github.com/apache/beam/pull/11163#discussion_r396766986 ## File path: sdks/python/apache_beam/runners/interactive/caching/streaming_cache.py

[GitHub] [beam] lostluck commented on a change in pull request #11188: [BEAM-3301] Adding restriction trackers and validation.

2020-03-23 Thread GitBox
lostluck commented on a change in pull request #11188: [BEAM-3301] Adding restriction trackers and validation. URL: https://github.com/apache/beam/pull/11188#discussion_r396795769 ## File path: sdks/go/pkg/beam/core/graph/fn_test.go ## @@ -676,39 +737,77 @@ func (fn *BadSd

[GitHub] [beam] lostluck commented on a change in pull request #11188: [BEAM-3301] Adding restriction trackers and validation.

2020-03-23 Thread GitBox
lostluck commented on a change in pull request #11188: [BEAM-3301] Adding restriction trackers and validation. URL: https://github.com/apache/beam/pull/11188#discussion_r396803249 ## File path: sdks/go/pkg/beam/core/graph/fn_test.go ## @@ -562,7 +595,13 @@ func (fn *GoodSd

[GitHub] [beam] lostluck commented on a change in pull request #11188: [BEAM-3301] Adding restriction trackers and validation.

2020-03-23 Thread GitBox
lostluck commented on a change in pull request #11188: [BEAM-3301] Adding restriction trackers and validation. URL: https://github.com/apache/beam/pull/11188#discussion_r396804143 ## File path: sdks/go/pkg/beam/core/sdf/sdf.go ## @@ -0,0 +1,74 @@ +// Licensed to the Apache

[GitHub] [beam] mxm commented on issue #11200: [BEAM-9573] Correct computing of watermark hold for timer output timestamp

2020-03-23 Thread GitBox
mxm commented on issue #11200: [BEAM-9573] Correct computing of watermark hold for timer output timestamp URL: https://github.com/apache/beam/pull/11200#issuecomment-602890165 Run Java PreCommit This is an automated message f

[GitHub] [beam] TheNeuralBit commented on a change in pull request #10529: [BEAM-9044] Protobuf options to Schema options

2020-03-23 Thread GitBox
TheNeuralBit commented on a change in pull request #10529: [BEAM-9044] Protobuf options to Schema options URL: https://github.com/apache/beam/pull/10529#discussion_r396791822 ## File path: sdks/java/extensions/protobuf/src/main/java/org/apache/beam/sdk/extensions/protobuf/ProtoSche

[GitHub] [beam] reuvenlax commented on issue #11074: Store logical type values in Row instead of base values

2020-03-23 Thread GitBox
reuvenlax commented on issue #11074: Store logical type values in Row instead of base values URL: https://github.com/apache/beam/pull/11074#issuecomment-602887176 Run Java PreCommit This is an automated message from the Apach

[GitHub] [beam] rohdesamuel opened a new pull request #11201: test

2020-03-23 Thread GitBox
rohdesamuel opened a new pull request #11201: test URL: https://github.com/apache/beam/pull/11201 Change-Id: I18a1a3c4589e1fdc90d919a223ef67465f495374 **Please** add a meaningful description for your change here Thank you for your contribution! F

[GitHub] [beam] boyuanzz commented on issue #11199: [BEAM-9562] Update Timer encoding to V2

2020-03-23 Thread GitBox
boyuanzz commented on issue #11199: [BEAM-9562] Update Timer encoding to V2 URL: https://github.com/apache/beam/pull/11199#issuecomment-602876259 I'll update the coder implementation once the proto looks good. This is an autom

[GitHub] [beam] pabloem merged pull request #11198: [BEAM-7923] Obfuscates display ids

2020-03-23 Thread GitBox
pabloem merged pull request #11198: [BEAM-7923] Obfuscates display ids URL: https://github.com/apache/beam/pull/11198 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu

[GitHub] [beam] mxm removed a comment on issue #11200: [BEAM-9573] Correct computing of watermark hold for timer output timestamp

2020-03-23 Thread GitBox
mxm removed a comment on issue #11200: [BEAM-9573] Correct computing of watermark hold for timer output timestamp URL: https://github.com/apache/beam/pull/11200#issuecomment-602869001 Flink Runner Nexmark Tests This is an aut

[GitHub] [beam] mxm commented on issue #11200: [BEAM-9573] Correct computing of watermark hold for timer output timestamp

2020-03-23 Thread GitBox
mxm commented on issue #11200: [BEAM-9573] Correct computing of watermark hold for timer output timestamp URL: https://github.com/apache/beam/pull/11200#issuecomment-602869187 Run Flink Runner Nexmark Tests This is an automat

[GitHub] [beam] mxm commented on issue #11200: [BEAM-9573] Correct computing of watermark hold for timer output timestamp

2020-03-23 Thread GitBox
mxm commented on issue #11200: [BEAM-9573] Correct computing of watermark hold for timer output timestamp URL: https://github.com/apache/beam/pull/11200#issuecomment-602869001 Flink Runner Nexmark Tests This is an automated m

[GitHub] [beam] mxm commented on issue #11200: [BEAM-9573] Correct computing of watermark hold for timer output timestamp

2020-03-23 Thread GitBox
mxm commented on issue #11200: [BEAM-9573] Correct computing of watermark hold for timer output timestamp URL: https://github.com/apache/beam/pull/11200#issuecomment-602866857 Run Java Flink PortableValidatesRunner Streaming

[GitHub] [beam] mxm commented on issue #11200: [BEAM-9573] Correct computing of watermark hold for timer output timestamp

2020-03-23 Thread GitBox
mxm commented on issue #11200: [BEAM-9573] Correct computing of watermark hold for timer output timestamp URL: https://github.com/apache/beam/pull/11200#issuecomment-602866787 Run Flink ValidatesRunner This is an automated me

[GitHub] [beam] mxm opened a new pull request #11200: [BEAM-9573] Correct computing of watermark hold for timer output timestamp

2020-03-23 Thread GitBox
mxm opened a new pull request #11200: [BEAM-9573] Correct computing of watermark hold for timer output timestamp URL: https://github.com/apache/beam/pull/11200 This PR contains two related changes which would be hard to review in separate PRs: ### BEAM-9573 Correct computing of wate

[GitHub] [beam] reuvenlax commented on issue #11074: Store logical type values in Row instead of base values

2020-03-23 Thread GitBox
reuvenlax commented on issue #11074: Store logical type values in Row instead of base values URL: https://github.com/apache/beam/pull/11074#issuecomment-602864720 Run Java PreCommit This is an automated message from the Apach

[GitHub] [beam] lostluck commented on issue #11197: [BEAM-8292] Portable Reshuffle for Go SDK

2020-03-23 Thread GitBox
lostluck commented on issue #11197: [BEAM-8292] Portable Reshuffle for Go SDK URL: https://github.com/apache/beam/pull/11197#issuecomment-602862121 Post commits run and pass which is a good sign! This is an automated message

[GitHub] [beam] chamikaramj commented on issue #11039: [BEAM-9383] Staging Dataflow artifacts from environment

2020-03-23 Thread GitBox
chamikaramj commented on issue #11039: [BEAM-9383] Staging Dataflow artifacts from environment URL: https://github.com/apache/beam/pull/11039#issuecomment-602855911 Retest this please This is an automated message from the Apa

[GitHub] [beam] chamikaramj commented on issue #11185: [BEAM-8019] Some generalizations to support cross-language transforms.

2020-03-23 Thread GitBox
chamikaramj commented on issue #11185: [BEAM-8019] Some generalizations to support cross-language transforms. URL: https://github.com/apache/beam/pull/11185#issuecomment-602855159 All tests pass now. Thanks. This is an automa

[GitHub] [beam] aaltay commented on issue #11174: [BEAM-7923] Pop failed transform when error is raised

2020-03-23 Thread GitBox
aaltay commented on issue #11174: [BEAM-7923] Pop failed transform when error is raised URL: https://github.com/apache/beam/pull/11174#issuecomment-602854440 retest this please This is an automated message from the Apache Git

[GitHub] [beam] robertwb commented on a change in pull request #11148: [BEAM-8335] Adds a streaming wordcount integration test

2020-03-23 Thread GitBox
robertwb commented on a change in pull request #11148: [BEAM-8335] Adds a streaming wordcount integration test URL: https://github.com/apache/beam/pull/11148#discussion_r396752081 ## File path: sdks/python/apache_beam/runners/interactive/interactive_runner_test.py ## @@ -

[GitHub] [beam] robertwb commented on a change in pull request #11148: [BEAM-8335] Adds a streaming wordcount integration test

2020-03-23 Thread GitBox
robertwb commented on a change in pull request #11148: [BEAM-8335] Adds a streaming wordcount integration test URL: https://github.com/apache/beam/pull/11148#discussion_r396751796 ## File path: sdks/python/apache_beam/runners/interactive/interactive_runner_test.py ## @@ -

[GitHub] [beam] robertwb commented on a change in pull request #11148: [BEAM-8335] Adds a streaming wordcount integration test

2020-03-23 Thread GitBox
robertwb commented on a change in pull request #11148: [BEAM-8335] Adds a streaming wordcount integration test URL: https://github.com/apache/beam/pull/11148#discussion_r396752506 ## File path: sdks/python/apache_beam/runners/interactive/interactive_runner_test.py ## @@ -

[GitHub] [beam] KevinGG commented on issue #11174: [BEAM-7923] Pop failed transform when error is raised

2020-03-23 Thread GitBox
KevinGG commented on issue #11174: [BEAM-7923] Pop failed transform when error is raised URL: https://github.com/apache/beam/pull/11174#issuecomment-602852406 Fixed a failed test. TL;DR, the test was wrong and passed in the past because it got lucky. The test used to pass and

[GitHub] [beam] lostluck commented on issue #11197: [BEAM-8292] Portable Reshuffle for Go SDK

2020-03-23 Thread GitBox
lostluck commented on issue #11197: [BEAM-8292] Portable Reshuffle for Go SDK URL: https://github.com/apache/beam/pull/11197#issuecomment-602851450 Run Go Postcommit This is an automated message from the Apache Git Service. To

[GitHub] [beam] aaltay commented on issue #11174: [BEAM-7923] Pop failed transform when error is raised

2020-03-23 Thread GitBox
aaltay commented on issue #11174: [BEAM-7923] Pop failed transform when error is raised URL: https://github.com/apache/beam/pull/11174#issuecomment-602851060 retest this please This is an automated message from the Apache Git

[GitHub] [beam] boyuanzz commented on a change in pull request #11060: [BEAM-9454] Create Deduplication transform based on user timer/state

2020-03-23 Thread GitBox
boyuanzz commented on a change in pull request #11060: [BEAM-9454] Create Deduplication transform based on user timer/state URL: https://github.com/apache/beam/pull/11060#discussion_r396747769 ## File path: sdks/python/apache_beam/transforms/deduplicate.py ## @@ -0,0 +1,13

[GitHub] [beam] pabloem commented on issue #11040: [BEAM-9305] Allow value provider query strings in _CustomBigQuerySource

2020-03-23 Thread GitBox
pabloem commented on issue #11040: [BEAM-9305] Allow value provider query strings in _CustomBigQuerySource URL: https://github.com/apache/beam/pull/11040#issuecomment-602847544 Thanks @EDjur for the contribution! Thanks @kamilwu for reviewing ---

[GitHub] [beam] pabloem merged pull request #11040: [BEAM-9305] Allow value provider query strings in _CustomBigQuerySource

2020-03-23 Thread GitBox
pabloem merged pull request #11040: [BEAM-9305] Allow value provider query strings in _CustomBigQuerySource URL: https://github.com/apache/beam/pull/11040 This is an automated message from the Apache Git Service. To respond

[GitHub] [beam] Hannah-Jiang merged pull request #11187: optionally import grpc

2020-03-23 Thread GitBox
Hannah-Jiang merged pull request #11187: optionally import grpc URL: https://github.com/apache/beam/pull/11187 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and u

[GitHub] [beam] KevinGG commented on a change in pull request #11174: [BEAM-7923] Pop failed transform when error is raised

2020-03-23 Thread GitBox
KevinGG commented on a change in pull request #11174: [BEAM-7923] Pop failed transform when error is raised URL: https://github.com/apache/beam/pull/11174#discussion_r396742548 ## File path: sdks/python/apache_beam/pipeline.py ## @@ -307,58 +307,61 @@ def _replace_if_neede

[GitHub] [beam] boyuanzz commented on a change in pull request #11060: [BEAM-9454] Create Deduplication transform based on user timer/state

2020-03-23 Thread GitBox
boyuanzz commented on a change in pull request #11060: [BEAM-9454] Create Deduplication transform based on user timer/state URL: https://github.com/apache/beam/pull/11060#discussion_r396742097 ## File path: sdks/python/apache_beam/transforms/deduplicate_test.py ## @@ -0,0

[GitHub] [beam] youngoli commented on issue #11188: [BEAM-3301] Adding restriction trackers and validation.

2020-03-23 Thread GitBox
youngoli commented on issue #11188: [BEAM-3301] Adding restriction trackers and validation. URL: https://github.com/apache/beam/pull/11188#issuecomment-602838126 Whoops, forgot reviewers. R: @lostluck This is an automated

[GitHub] [beam] pabloem commented on issue #11198: [BEAM-7923] Obfuscates display ids

2020-03-23 Thread GitBox
pabloem commented on issue #11198: [BEAM-7923] Obfuscates display ids URL: https://github.com/apache/beam/pull/11198#issuecomment-602837778 retest this please This is an automated message from the Apache Git Service. To respon

[GitHub] [beam] lostluck commented on issue #11197: [BEAM-8292] Portable Reshuffle for Go SDK

2020-03-23 Thread GitBox
lostluck commented on issue #11197: [BEAM-8292] Portable Reshuffle for Go SDK URL: https://github.com/apache/beam/pull/11197#issuecomment-602835398 Retest this please This is an automated message from the Apache Git Service. T

[GitHub] [beam] pabloem commented on issue #10291: [BEAM-7516][BEAM-8823] FnApiRunner works with work queues, and a primitive watermark manager

2020-03-23 Thread GitBox
pabloem commented on issue #10291: [BEAM-7516][BEAM-8823] FnApiRunner works with work queues, and a primitive watermark manager URL: https://github.com/apache/beam/pull/10291#issuecomment-602830331 I can't reproduce any of the failures locally via `tox -e py35-cloud,py35-cython` -

[GitHub] [beam] boyuanzz opened a new pull request #11199: [BEAM-9562] Update Timer encoding to V2

2020-03-23 Thread GitBox
boyuanzz opened a new pull request #11199: [BEAM-9562] Update Timer encoding to V2 URL: https://github.com/apache/beam/pull/11199 **Please** add a meaningful description for your change here Thank you for your contribution! Follow this checklist to hel

[GitHub] [beam] rohdesamuel commented on issue #11148: [BEAM-8335] Adds a streaming wordcount integration test

2020-03-23 Thread GitBox
rohdesamuel commented on issue #11148: [BEAM-8335] Adds a streaming wordcount integration test URL: https://github.com/apache/beam/pull/11148#issuecomment-602825559 R: @robertwb This is an automated message from the Apache G

[GitHub] [beam] reuvenlax commented on issue #11074: Store logical type values in Row instead of base values

2020-03-23 Thread GitBox
reuvenlax commented on issue #11074: Store logical type values in Row instead of base values URL: https://github.com/apache/beam/pull/11074#issuecomment-602824816 Run Java PreCommit This is an automated message from the Apach

[GitHub] [beam] udim commented on issue #10914: [BEAM-8078] streaming_wordcount_debugging.py is missing a test

2020-03-23 Thread GitBox
udim commented on issue #10914: [BEAM-8078] streaming_wordcount_debugging.py is missing a test URL: https://github.com/apache/beam/pull/10914#issuecomment-602823472 run python 3.7 postcommit This is an automated message from

[GitHub] [beam] udim commented on issue #10914: [BEAM-8078] streaming_wordcount_debugging.py is missing a test

2020-03-23 Thread GitBox
udim commented on issue #10914: [BEAM-8078] streaming_wordcount_debugging.py is missing a test URL: https://github.com/apache/beam/pull/10914#issuecomment-602823166 Trying again This is an automated message from the Apache Gi

[GitHub] [beam] chamikaramj commented on issue #11039: [BEAM-9383] Staging Dataflow artifacts from environment

2020-03-23 Thread GitBox
chamikaramj commented on issue #11039: [BEAM-9383] Staging Dataflow artifacts from environment URL: https://github.com/apache/beam/pull/11039#issuecomment-602822540 Retest this please This is an automated message from the Apa

[GitHub] [beam] chamikaramj commented on issue #11185: [BEAM-8019] Some generalizations to support cross-language transforms.

2020-03-23 Thread GitBox
chamikaramj commented on issue #11185: [BEAM-8019] Some generalizations to support cross-language transforms. URL: https://github.com/apache/beam/pull/11185#issuecomment-602821407 Run Python2_PVR_Flink PreCommit This is an au

[GitHub] [beam] davidyan74 commented on a change in pull request #11198: [BEAM-7923] Obfuscates display ids

2020-03-23 Thread GitBox
davidyan74 commented on a change in pull request #11198: [BEAM-7923] Obfuscates display ids URL: https://github.com/apache/beam/pull/11198#discussion_r396717186 ## File path: sdks/python/apache_beam/runners/interactive/display/pcoll_visualization.py ## @@ -246,11 +247,11

[GitHub] [beam] pabloem commented on issue #11198: [BEAM-7923] Obfuscates display ids

2020-03-23 Thread GitBox
pabloem commented on issue #11198: [BEAM-7923] Obfuscates display ids URL: https://github.com/apache/beam/pull/11198#issuecomment-602819811 retest this please This is an automated message from the Apache Git Service. To respon

[GitHub] [beam] lostluck commented on issue #11197: [BEAM-8292] Portable Reshuffle for Go SDK

2020-03-23 Thread GitBox
lostluck commented on issue #11197: [BEAM-8292] Portable Reshuffle for Go SDK URL: https://github.com/apache/beam/pull/11197#issuecomment-602817069 I'm definitely not merging this until both the PostCommit runs, and someone more familiar with windowing/trigger semantics looks over the confi

[GitHub] [beam] KevinGG opened a new pull request #11198: [BEAM-7923] Obfuscates display ids

2020-03-23 Thread GitBox
KevinGG opened a new pull request #11198: [BEAM-7923] Obfuscates display ids URL: https://github.com/apache/beam/pull/11198 1. Use md5 to hash and digest any inputs into a hexadecimal string. 2. The obfuscation is applied to all display ids in notebooks. Note the ids will start with al

[GitHub] [beam] KevinGG commented on issue #11198: [BEAM-7923] Obfuscates display ids

2020-03-23 Thread GitBox
KevinGG commented on issue #11198: [BEAM-7923] Obfuscates display ids URL: https://github.com/apache/beam/pull/11198#issuecomment-602817070 yapf formatted. Lint passed locally. R: @pabloem R: @davidyan74 R: @rohdesamuel PTAL, thx! --

[GitHub] [beam] lukecwik merged pull request #11192: [BEAM-9430] Fix coder sent to Dataflow service for non-portable pipelines due to WatermarkEstimators migration change

2020-03-23 Thread GitBox
lukecwik merged pull request #11192: [BEAM-9430] Fix coder sent to Dataflow service for non-portable pipelines due to WatermarkEstimators migration change URL: https://github.com/apache/beam/pull/11192 This is an automated m

[GitHub] [beam] reuvenlax merged pull request #10990: [BEAM-9569] disable coder inference for rows

2020-03-23 Thread GitBox
reuvenlax merged pull request #10990: [BEAM-9569] disable coder inference for rows URL: https://github.com/apache/beam/pull/10990 This is an automated message from the Apache Git Service. To respond to the message, please lo

  1   2   3   4   5   6   7   8   9   10   >