[GitHub] [beam] yifanmai commented on pull request #12656: [BEAM-10781] Add PTransformOverride.get_replacement_transform_for_applied_ptransform()

2020-09-15 Thread GitBox
yifanmai commented on pull request #12656: URL: https://github.com/apache/beam/pull/12656#issuecomment-692808999 Awesome, thanks for your help! This is an automated message from the Apache Git Service. To respond to the

[GitHub] [beam] yifanmai commented on pull request #12656: [BEAM-10781] Add PTransformOverride.get_replacement_transform_for_applied_ptransform()

2020-09-14 Thread GitBox
yifanmai commented on pull request #12656: URL: https://github.com/apache/beam/pull/12656#issuecomment-692172601 The failures are on `DataChannelTest.test_time_based_flush_grpc_data_channel` which does not seem related to the PR. @chamikaramj is this ready for merge or is there more

[GitHub] [beam] yifanmai commented on pull request #12656: [BEAM-10781] Add PTransformOverride.get_replacement_transform_for_applied_ptransform()

2020-09-12 Thread GitBox
yifanmai commented on pull request #12656: URL: https://github.com/apache/beam/pull/12656#issuecomment-690835921 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [beam] yifanmai commented on pull request #12656: [BEAM-10781] Add PTransformOverride.get_replacement_transform_for_applied_ptransform()

2020-09-12 Thread GitBox
yifanmai commented on pull request #12656: URL: https://github.com/apache/beam/pull/12656#issuecomment-690835921 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [beam] yifanmai commented on pull request #12656: [BEAM-10781] Add PTransformOverride.get_replacement_transform_for_applied_ptransform()

2020-09-11 Thread GitBox
yifanmai commented on pull request #12656: URL: https://github.com/apache/beam/pull/12656#issuecomment-691274764 3.5 PostCommit looks good This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [beam] yifanmai commented on pull request #12656: [BEAM-10781] Add PTransformOverride.get_replacement_transform_for_applied_ptransform()

2020-09-11 Thread GitBox
yifanmai commented on pull request #12656: URL: https://github.com/apache/beam/pull/12656#issuecomment-691238940 @chamikaramj could you re-run post submits? I fixed a typo that was causing the breakage. This is an automated

[GitHub] [beam] yifanmai commented on pull request #12656: [BEAM-10781] Add PTransformOverride.get_replacement_transform_for_applied_ptransform()

2020-09-10 Thread GitBox
yifanmai commented on pull request #12656: URL: https://github.com/apache/beam/pull/12656#issuecomment-690835921 Run Python 3.5 PostCommit This is an automated message from the Apache Git Service. To respond to the

[GitHub] [beam] yifanmai commented on pull request #12656: [BEAM-10781] Add PTransformOverride.get_replacement_transform_for_applied_ptransform()

2020-09-10 Thread GitBox
yifanmai commented on pull request #12656: URL: https://github.com/apache/beam/pull/12656#issuecomment-690780411 Run Python PreCommit This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [beam] yifanmai commented on pull request #12656: [BEAM-10781] Add PTransformOverride.get_replacement_transform_for_applied_ptransform()

2020-09-09 Thread GitBox
yifanmai commented on pull request #12656: URL: https://github.com/apache/beam/pull/12656#issuecomment-689841127 @chamikaramj Can this be merged? This is an automated message from the Apache Git Service. To respond to the

[GitHub] [beam] yifanmai commented on pull request #12656: [BEAM-10781] Add PTransformOverride.get_replacement_transform_for_applied_ptransform()

2020-09-08 Thread GitBox
yifanmai commented on pull request #12656: URL: https://github.com/apache/beam/pull/12656#issuecomment-689157882 Precommits (besides codecov) look good. This is an automated message from the Apache Git Service. To respond to

[GitHub] [beam] yifanmai commented on pull request #12656: [BEAM-10781] Add PTransformOverride.get_replacement_transform_for_applied_ptransform()

2020-08-31 Thread GitBox
yifanmai commented on pull request #12656: URL: https://github.com/apache/beam/pull/12656#issuecomment-684001855 I seem to be running into the same issues as in #12655 with failing tests; how should I resolve this? This is

[GitHub] [beam] yifanmai commented on pull request #12656: [BEAM-10781] Add PTransformOverride.get_replacement_transform_for_applied_ptransform()

2020-08-20 Thread GitBox
yifanmai commented on pull request #12656: URL: https://github.com/apache/beam/pull/12656#issuecomment-678012887 R: @chamikaramj This is an automated message from the Apache Git Service. To respond to the message, please log