[GitHub] [beam] mosche commented on pull request #16607: [BEAM-13245] Generalize S3FileSystem to support multiple URI schemes.

2022-01-27 Thread GitBox
mosche commented on pull request #16607: URL: https://github.com/apache/beam/pull/16607#issuecomment-1022955583 Run Java PreCommit -- 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 the specific co

[GitHub] [beam] mosche commented on pull request #16607: [BEAM-13245] Generalize S3FileSystem to support multiple URI schemes.

2022-01-27 Thread GitBox
mosche commented on pull request #16607: URL: https://github.com/apache/beam/pull/16607#issuecomment-1022973109 Run Java PreCommit -- 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 the specific co

[GitHub] [beam] aromanenko-dev commented on pull request #16579: Revert "Revert "Merge pull request #15863 from [BEAM-13184] Autoshard…

2022-01-27 Thread GitBox
aromanenko-dev commented on pull request #16579: URL: https://github.com/apache/beam/pull/16579#issuecomment-1022976574 Run Java PostCommit -- 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 the sp

[GitHub] [beam] codecov[bot] edited a comment on pull request #16605: [BEAM-13634][Playground] Create a separate Dockerfile for the routing service

2022-01-27 Thread GitBox
codecov[bot] edited a comment on pull request #16605: URL: https://github.com/apache/beam/pull/16605#issuecomment-1020948543 # [Codecov](https://codecov.io/gh/apache/beam/pull/16605?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+

[GitHub] [beam] mosche commented on a change in pull request #16607: [BEAM-13245] Generalize S3FileSystem to support multiple URI schemes.

2022-01-27 Thread GitBox
mosche commented on a change in pull request #16607: URL: https://github.com/apache/beam/pull/16607#discussion_r793412422 ## File path: sdks/java/io/amazon-web-services2/src/test/java/org/apache/beam/sdk/io/aws2/s3/S3ResourceIdTest.java ## @@ -75,6 +81,62 @@ new Tes

[GitHub] [beam] KhaninArtur commented on a change in pull request #16618: [BEAM-13744] [Playground] Add context line field to the tags

2022-01-27 Thread GitBox
KhaninArtur commented on a change in pull request #16618: URL: https://github.com/apache/beam/pull/16618#discussion_r793421440 ## File path: website/www/site/content/en/get-started/try-beam-playground.md ## @@ -80,7 +86,9 @@ The 'beam-playground' tag consists of the following

[GitHub] [beam] ashb commented on a change in pull request #16511: [BEAM-12812] - Run Github Actions on GCP workers in the apache-beam-testing project

2022-01-27 Thread GitBox
ashb commented on a change in pull request #16511: URL: https://github.com/apache/beam/pull/16511#discussion_r793463559 ## File path: .github/dockerized-gh-actions-runners/self-hosted-linux/docker/entrypoint.sh ## @@ -0,0 +1,46 @@ +#!/usr/bin/env bash + +# Licensed to the Apac

[GitHub] [beam] mosche opened a new pull request #16628: [adhoc] Test S3Options and AwsOptions for Sdk v2

2022-01-27 Thread GitBox
mosche opened a new pull request #16628: URL: https://github.com/apache/beam/pull/16628 Add basic tests for S3Options and AwsOptions. @aromanenko-dev Btw, the reason the issue in https://github.com/apache/beam/pull/16607 didn't surface earlier is that factories in PipelineOptions ar

[GitHub] [beam] mosche commented on pull request #16628: [adhoc] Test S3Options and AwsOptions for Sdk v2

2022-01-27 Thread GitBox
mosche commented on pull request #16628: URL: https://github.com/apache/beam/pull/16628#issuecomment-1023102448 R: @aromanenko-dev -- 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 the specific c

[GitHub] [beam] codecov[bot] edited a comment on pull request #16618: [BEAM-13744] [Playground] Add context line field to the tags

2022-01-27 Thread GitBox
codecov[bot] edited a comment on pull request #16618: URL: https://github.com/apache/beam/pull/16618#issuecomment-1021947095 # [Codecov](https://codecov.io/gh/apache/beam/pull/16618?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+

[GitHub] [beam] pavel-avilov commented on a change in pull request #16618: [BEAM-13744] [Playground] Add context line field to the tags

2022-01-27 Thread GitBox
pavel-avilov commented on a change in pull request #16618: URL: https://github.com/apache/beam/pull/16618#discussion_r793526055 ## File path: website/www/site/content/en/get-started/try-beam-playground.md ## @@ -80,7 +86,9 @@ The 'beam-playground' tag consists of the following

[GitHub] [beam] codecov[bot] edited a comment on pull request #16618: [BEAM-13744] [Playground] Add context line field to the tags

2022-01-27 Thread GitBox
codecov[bot] edited a comment on pull request #16618: URL: https://github.com/apache/beam/pull/16618#issuecomment-1021947095 -- 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 the specific comment.

[GitHub] [beam] codecov[bot] edited a comment on pull request #16618: [BEAM-13744] [Playground] Add context line field to the tags

2022-01-27 Thread GitBox
codecov[bot] edited a comment on pull request #16618: URL: https://github.com/apache/beam/pull/16618#issuecomment-1021947095 # [Codecov](https://codecov.io/gh/apache/beam/pull/16618?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+

[GitHub] [beam] codecov[bot] edited a comment on pull request #16618: [BEAM-13744] [Playground] Add context line field to the tags

2022-01-27 Thread GitBox
codecov[bot] edited a comment on pull request #16618: URL: https://github.com/apache/beam/pull/16618#issuecomment-1021947095 # [Codecov](https://codecov.io/gh/apache/beam/pull/16618?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+

[GitHub] [beam] codecov[bot] edited a comment on pull request #16618: [BEAM-13744] [Playground] Add context line field to the tags

2022-01-27 Thread GitBox
codecov[bot] edited a comment on pull request #16618: URL: https://github.com/apache/beam/pull/16618#issuecomment-1021947095 # [Codecov](https://codecov.io/gh/apache/beam/pull/16618?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+

[GitHub] [beam] codecov[bot] edited a comment on pull request #16618: [BEAM-13744] [Playground] Add context line field to the tags

2022-01-27 Thread GitBox
codecov[bot] edited a comment on pull request #16618: URL: https://github.com/apache/beam/pull/16618#issuecomment-1021947095 # [Codecov](https://codecov.io/gh/apache/beam/pull/16618?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+

[GitHub] [beam] codecov[bot] edited a comment on pull request #16618: [BEAM-13744] [Playground] Add context line field to the tags

2022-01-27 Thread GitBox
codecov[bot] edited a comment on pull request #16618: URL: https://github.com/apache/beam/pull/16618#issuecomment-1021947095 # [Codecov](https://codecov.io/gh/apache/beam/pull/16618?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+

[GitHub] [beam] codecov[bot] edited a comment on pull request #16618: [BEAM-13744] [Playground] Add context line field to the tags

2022-01-27 Thread GitBox
codecov[bot] edited a comment on pull request #16618: URL: https://github.com/apache/beam/pull/16618#issuecomment-1021947095 # [Codecov](https://codecov.io/gh/apache/beam/pull/16618?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+

[GitHub] [beam] codecov[bot] edited a comment on pull request #16618: [BEAM-13744] [Playground] Add context line field to the tags

2022-01-27 Thread GitBox
codecov[bot] edited a comment on pull request #16618: URL: https://github.com/apache/beam/pull/16618#issuecomment-1021947095 # [Codecov](https://codecov.io/gh/apache/beam/pull/16618?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+

[GitHub] [beam] codecov[bot] edited a comment on pull request #16618: [BEAM-13744] [Playground] Add context line field to the tags

2022-01-27 Thread GitBox
codecov[bot] edited a comment on pull request #16618: URL: https://github.com/apache/beam/pull/16618#issuecomment-1021947095 # [Codecov](https://codecov.io/gh/apache/beam/pull/16618?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+

[GitHub] [beam] codecov[bot] edited a comment on pull request #16618: [BEAM-13744] [Playground] Add context line field to the tags

2022-01-27 Thread GitBox
codecov[bot] edited a comment on pull request #16618: URL: https://github.com/apache/beam/pull/16618#issuecomment-1021947095 # [Codecov](https://codecov.io/gh/apache/beam/pull/16618?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+

[GitHub] [beam] codecov[bot] edited a comment on pull request #16618: [BEAM-13744] [Playground] Add context line field to the tags

2022-01-27 Thread GitBox
codecov[bot] edited a comment on pull request #16618: URL: https://github.com/apache/beam/pull/16618#issuecomment-1021947095 # [Codecov](https://codecov.io/gh/apache/beam/pull/16618?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+

[GitHub] [beam] codecov[bot] edited a comment on pull request #16618: [BEAM-13744] [Playground] Add context line field to the tags

2022-01-27 Thread GitBox
codecov[bot] edited a comment on pull request #16618: URL: https://github.com/apache/beam/pull/16618#issuecomment-1021947095 # [Codecov](https://codecov.io/gh/apache/beam/pull/16618?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+

[GitHub] [beam] codecov[bot] edited a comment on pull request #16618: [BEAM-13744] [Playground] Add context line field to the tags

2022-01-27 Thread GitBox
codecov[bot] edited a comment on pull request #16618: URL: https://github.com/apache/beam/pull/16618#issuecomment-1021947095 # [Codecov](https://codecov.io/gh/apache/beam/pull/16618?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+

[GitHub] [beam] mosche commented on pull request #16628: [adhoc] Test S3Options and AwsOptions for Sdk v2

2022-01-27 Thread GitBox
mosche commented on pull request #16628: URL: https://github.com/apache/beam/pull/16628#issuecomment-1023146485 Run Java PreCommit -- 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 the specific co

[GitHub] [beam] codecov[bot] edited a comment on pull request #16618: [BEAM-13744] [Playground] Add context line field to the tags

2022-01-27 Thread GitBox
codecov[bot] edited a comment on pull request #16618: URL: https://github.com/apache/beam/pull/16618#issuecomment-1021947095 # [Codecov](https://codecov.io/gh/apache/beam/pull/16618?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+

[GitHub] [beam] pavel-avilov commented on a change in pull request #16617: [BEAM-13743] [Playground] Add context line for examples

2022-01-27 Thread GitBox
pavel-avilov commented on a change in pull request #16617: URL: https://github.com/apache/beam/pull/16617#discussion_r793551523 ## File path: playground/infrastructure/helper.py ## @@ -331,6 +333,17 @@ def _validate(tag: dict, supported_categories: List[str]) -> bool:

[GitHub] [beam] codecov[bot] edited a comment on pull request #16618: [BEAM-13744] [Playground] Add context line field to the tags

2022-01-27 Thread GitBox
codecov[bot] edited a comment on pull request #16618: URL: https://github.com/apache/beam/pull/16618#issuecomment-1021947095 # [Codecov](https://codecov.io/gh/apache/beam/pull/16618?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+

[GitHub] [beam] daria-malkova commented on pull request #16617: [BEAM-13743] [Playground] Add context line for examples

2022-01-27 Thread GitBox
daria-malkova commented on pull request #16617: URL: https://github.com/apache/beam/pull/16617#issuecomment-1023223406 LGTM -- 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 the specific comment.

[GitHub] [beam] yeandy commented on pull request #16590: [BEAM-13605] Update pandas_doctests_test denylists in preparation for pandas 1.4.0

2022-01-27 Thread GitBox
yeandy commented on pull request #16590: URL: https://github.com/apache/beam/pull/16590#issuecomment-1023272908 Fixed the backwards compatibility issue, and it now passes Python 3.6 and 3.7 now. The PR is now in a state for your review and/or other additions. -- This is an automated mes

[GitHub] [beam] mattrudary commented on a change in pull request #16607: [BEAM-13245] Generalize S3FileSystem to support multiple URI schemes.

2022-01-27 Thread GitBox
mattrudary commented on a change in pull request #16607: URL: https://github.com/apache/beam/pull/16607#discussion_r793673671 ## File path: sdks/java/io/amazon-web-services2/src/test/java/org/apache/beam/sdk/io/aws2/s3/S3ResourceIdTest.java ## @@ -75,6 +81,62 @@ new

[GitHub] [beam] mosche commented on a change in pull request #16607: [BEAM-13245] Generalize S3FileSystem to support multiple URI schemes.

2022-01-27 Thread GitBox
mosche commented on a change in pull request #16607: URL: https://github.com/apache/beam/pull/16607#discussion_r793731429 ## File path: sdks/java/io/amazon-web-services2/src/test/java/org/apache/beam/sdk/io/aws2/s3/S3ResourceIdTest.java ## @@ -75,6 +81,62 @@ new Tes

[GitHub] [beam] pabloem commented on pull request #16624: [BEAM-13749] Exclude per-key order tests on Twister2 runner

2022-01-27 Thread GitBox
pabloem commented on pull request #16624: URL: https://github.com/apache/beam/pull/16624#issuecomment-1023421007 Retest this please -- 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 the specific c

[GitHub] [beam] pabloem commented on pull request #16613: Supporting JdbcIO driver in classpath for x-lang

2022-01-27 Thread GitBox
pabloem commented on pull request #16613: URL: https://github.com/apache/beam/pull/16613#issuecomment-1023421745 Run Java PreCommit -- 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 the specific c

[GitHub] [beam] pabloem commented on pull request #16624: [BEAM-13749] Exclude per-key order tests on Twister2 runner

2022-01-27 Thread GitBox
pabloem commented on pull request #16624: URL: https://github.com/apache/beam/pull/16624#issuecomment-1023426614 Run Java PostCommit -- 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 the specific

[GitHub] [beam] kennknowles commented on a change in pull request #16625: [BEAM-13537] Fix NPE in kafkatopubsub example

2022-01-27 Thread GitBox
kennknowles commented on a change in pull request #16625: URL: https://github.com/apache/beam/pull/16625#discussion_r793803248 ## File path: examples/java/src/main/java/org/apache/beam/examples/complete/kafkatopubsub/options/KafkaToPubsubOptions.java ## @@ -90,6 +91,7 @@

[GitHub] [beam] codecov[bot] edited a comment on pull request #16613: Supporting JdbcIO driver in classpath for x-lang

2022-01-27 Thread GitBox
codecov[bot] edited a comment on pull request #16613: URL: https://github.com/apache/beam/pull/16613#issuecomment-1021658774 # [Codecov](https://codecov.io/gh/apache/beam/pull/16613?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+

[GitHub] [beam] TheNeuralBit commented on pull request #16625: [BEAM-13537] Fix NPE in kafkatopubsub example

2022-01-27 Thread GitBox
TheNeuralBit commented on pull request #16625: URL: https://github.com/apache/beam/pull/16625#issuecomment-1023440669 Run Java PreCommit -- 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 the speci

[GitHub] [beam] TheNeuralBit commented on pull request #16625: [BEAM-13537] Fix NPE in kafkatopubsub example

2022-01-27 Thread GitBox
TheNeuralBit commented on pull request #16625: URL: https://github.com/apache/beam/pull/16625#issuecomment-1023440789 Run Java_Examples_Dataflow PreCommit -- 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

[GitHub] [beam] ilya-kozyrev commented on pull request #16605: [BEAM-13634][Playground] Create a separate Dockerfile for the routing service

2022-01-27 Thread GitBox
ilya-kozyrev commented on pull request #16605: URL: https://github.com/apache/beam/pull/16605#issuecomment-1023466967 Run Java PreCommit -- 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 the speci

[GitHub] [beam] ilya-kozyrev commented on pull request #16605: [BEAM-13634][Playground] Create a separate Dockerfile for the routing service

2022-01-27 Thread GitBox
ilya-kozyrev commented on pull request #16605: URL: https://github.com/apache/beam/pull/16605#issuecomment-1023467145 Run Python PreCommit -- 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 the spe

[GitHub] [beam] pabloem commented on pull request #16526: [WIP][BEAM-13051][C] Enable pylint warnings (misplaced-bare-raise/no-member)

2022-01-27 Thread GitBox
pabloem commented on pull request #16526: URL: https://github.com/apache/beam/pull/16526#issuecomment-1023468058 Run Python PreCommit -- 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 the specific

[GitHub] [beam] riteshghorse opened a new pull request #16629: [BEAM-13757] adds pane observation in DoFn

2022-01-27 Thread GitBox
riteshghorse opened a new pull request #16629: URL: https://github.com/apache/beam/pull/16629 Resolves [BEAM-13757](https://issues.apache.org/jira/browse/BEAM-13757). Functionality for users to extract PaneInfo in DoFns. Thank you for your contribut

[GitHub] [beam] TheNeuralBit commented on a change in pull request #16625: [BEAM-13537] Fix NPE in kafkatopubsub example

2022-01-27 Thread GitBox
TheNeuralBit commented on a change in pull request #16625: URL: https://github.com/apache/beam/pull/16625#discussion_r793855054 ## File path: examples/java/src/main/java/org/apache/beam/examples/complete/kafkatopubsub/options/KafkaToPubsubOptions.java ## @@ -90,6 +91,7 @@

[GitHub] [beam] TheNeuralBit merged pull request #16625: [BEAM-13537] Fix NPE in kafkatopubsub example

2022-01-27 Thread GitBox
TheNeuralBit merged pull request #16625: URL: https://github.com/apache/beam/pull/16625 -- 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 the specific comment. To unsubscribe, e-mail: github-uns

[GitHub] [beam] codecov[bot] commented on pull request #16629: [BEAM-13757] adds pane observation in DoFn

2022-01-27 Thread GitBox
codecov[bot] commented on pull request #16629: URL: https://github.com/apache/beam/pull/16629#issuecomment-1023486045 # [Codecov](https://codecov.io/gh/apache/beam/pull/16629?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+

[GitHub] [beam] codecov[bot] edited a comment on pull request #16629: [BEAM-13757] adds pane observation in DoFn

2022-01-27 Thread GitBox
codecov[bot] edited a comment on pull request #16629: URL: https://github.com/apache/beam/pull/16629#issuecomment-1023486045 # [Codecov](https://codecov.io/gh/apache/beam/pull/16629?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+

[GitHub] [beam] codecov[bot] edited a comment on pull request #16629: [BEAM-13757] adds pane observation in DoFn

2022-01-27 Thread GitBox
codecov[bot] edited a comment on pull request #16629: URL: https://github.com/apache/beam/pull/16629#issuecomment-1023486045 # [Codecov](https://codecov.io/gh/apache/beam/pull/16629?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+

[GitHub] [beam] aromanenko-dev merged pull request #16628: [adhoc] Test S3Options and AwsOptions for Sdk v2

2022-01-27 Thread GitBox
aromanenko-dev merged pull request #16628: URL: https://github.com/apache/beam/pull/16628 -- 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 the specific comment. To unsubscribe, e-mail: github-u

[GitHub] [beam] codecov[bot] edited a comment on pull request #16629: [BEAM-13757] adds pane observation in DoFn

2022-01-27 Thread GitBox
codecov[bot] edited a comment on pull request #16629: URL: https://github.com/apache/beam/pull/16629#issuecomment-1023486045 # [Codecov](https://codecov.io/gh/apache/beam/pull/16629?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+

[GitHub] [beam] emilymye opened a new pull request #16630: Update Python SDK beam master container tags

2022-01-27 Thread GitBox
emilymye opened a new pull request #16630: URL: https://github.com/apache/beam/pull/16630 Includes minor bumps to dependencies R: @tvalentyn Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly

[GitHub] [beam] kileys opened a new pull request #16631: [BEAM-13741] Switch sql dependency to provided

2022-01-27 Thread GitBox
kileys opened a new pull request #16631: URL: https://github.com/apache/beam/pull/16631 :sdks:java:extensions:sql provides vendored calcite Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easi

[GitHub] [beam] codecov[bot] edited a comment on pull request #14355: [WIP][BEAM-11976] Bump org.apache.kafka:kafka_2.12 to 2.7.0

2022-01-27 Thread GitBox
codecov[bot] edited a comment on pull request #14355: URL: https://github.com/apache/beam/pull/14355#issuecomment-809510122 # [Codecov](https://codecov.io/gh/apache/beam/pull/14355?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+A

[GitHub] [beam] kileys commented on pull request #16631: [BEAM-13741] Switch sql dependency to provided

2022-01-27 Thread GitBox
kileys commented on pull request #16631: URL: https://github.com/apache/beam/pull/16631#issuecomment-1023502822 ./gradlew publish -- 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 the specific com

[GitHub] [beam] riteshghorse commented on pull request #16629: [BEAM-13757] adds pane observation in DoFn

2022-01-27 Thread GitBox
riteshghorse commented on pull request #16629: URL: https://github.com/apache/beam/pull/16629#issuecomment-1023507438 R: @lostluck -- 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 the specific c

[GitHub] [beam] kileys commented on pull request #16631: [BEAM-13741] Switch sql dependency to provided

2022-01-27 Thread GitBox
kileys commented on pull request #16631: URL: https://github.com/apache/beam/pull/16631#issuecomment-1023507338 Run Gradle Publish -- 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 the specific co

[GitHub] [beam] emilymye opened a new pull request #16632: [DO NOT MERGE] Run Python RC Validation Tests

2022-01-27 Thread GitBox
emilymye opened a new pull request #16632: URL: https://github.com/apache/beam/pull/16632 Run Python ReleaseCandidate Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily: - [ ] [**C

[GitHub] [beam] emilymye commented on pull request #16632: [DO NOT MERGE] Run Python RC Validation Tests

2022-01-27 Thread GitBox
emilymye commented on pull request #16632: URL: https://github.com/apache/beam/pull/16632#issuecomment-1023513800 Run Python ReleaseCandidate -- 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 the

[GitHub] [beam] codecov[bot] commented on pull request #16632: [DO NOT MERGE] Run Python RC Validation Tests

2022-01-27 Thread GitBox
codecov[bot] commented on pull request #16632: URL: https://github.com/apache/beam/pull/16632#issuecomment-1023518593 # [Codecov](https://codecov.io/gh/apache/beam/pull/16632?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+

[GitHub] [beam] AnandInguva opened a new pull request #16633: [BEAM-4032]Support staging binary distributions of dependency packages

2022-01-27 Thread GitBox
AnandInguva opened a new pull request #16633: URL: https://github.com/apache/beam/pull/16633 **Please** add a meaningful description for your change here Thank you for your contribution! Follow this checklist to help us incorporate your contribution qu

[GitHub] [beam] y1chi closed pull request #16219: [BEAM-13193] Implement ProcessBundleResponse Elements embedding in Ja…

2022-01-27 Thread GitBox
y1chi closed pull request #16219: URL: https://github.com/apache/beam/pull/16219 -- 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 the specific comment. To unsubscribe, e-mail: github-unsubscr..

[GitHub] [beam] mouyang commented on pull request #14350: [BEAM-9613] fix handling of BQ BOOLEAN, FLOAT, REPEATED

2022-01-27 Thread GitBox
mouyang commented on pull request #14350: URL: https://github.com/apache/beam/pull/14350#issuecomment-1023533056 @aaltay I haven't had a chance to do further work on this and I'm not sure when I will get the chance, but I definitely think this needs to be addressed because it's definitely

[GitHub] [beam] codecov[bot] edited a comment on pull request #16526: [WIP][BEAM-13051][C] Enable pylint warnings (misplaced-bare-raise/no-member)

2022-01-27 Thread GitBox
codecov[bot] edited a comment on pull request #16526: URL: https://github.com/apache/beam/pull/16526#issuecomment-1013509394 # [Codecov](https://codecov.io/gh/apache/beam/pull/16526?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+

[GitHub] [beam] lostluck commented on pull request #16608: [BEAM-13740] update java_tests.yml to temporarily remove setup-go

2022-01-27 Thread GitBox
lostluck commented on pull request #16608: URL: https://github.com/apache/beam/pull/16608#issuecomment-1023535806 Going to merge this in, but we may still run into failures related to Go in the Actions, assuming there's no go involved. The ultimate fix is probably to just duplicate the set

[GitHub] [beam] lostluck merged pull request #16608: [BEAM-13740] update java_tests.yml to temporarily remove setup-go

2022-01-27 Thread GitBox
lostluck merged pull request #16608: URL: https://github.com/apache/beam/pull/16608 -- 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 the specific comment. To unsubscribe, e-mail: github-unsubsc

[GitHub] [beam] codecov[bot] edited a comment on pull request #16521: [WIP][BEAM-13051][A] Enable pylint warnings (function-redefined/bad-super-call)

2022-01-27 Thread GitBox
codecov[bot] edited a comment on pull request #16521: URL: https://github.com/apache/beam/pull/16521#issuecomment-1013397280 # [Codecov](https://codecov.io/gh/apache/beam/pull/16521?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+

[GitHub] [beam] lostluck commented on a change in pull request #16629: [BEAM-13757] adds pane observation in DoFn

2022-01-27 Thread GitBox
lostluck commented on a change in pull request #16629: URL: https://github.com/apache/beam/pull/16629#discussion_r793879221 ## File path: sdks/go/pkg/beam/core/funcx/fn.go ## @@ -17,9 +17,10 @@ package funcx import ( "fmt" - "github.com/apache/beam/sdks/v2/go/p

[GitHub] [beam] MarcoRob opened a new pull request #16634: [BEAM-8218] PulsarIO Connector

2022-01-27 Thread GitBox
MarcoRob opened a new pull request #16634: URL: https://github.com/apache/beam/pull/16634 PulsarIO Connector first version Contains: - Pulsar Reader (SDF based) - Pulsar Writer - Pulsar Transform - Unit tests - IT tests with Pulsar container

[GitHub] [beam] MarcoRob commented on pull request #15572: [BEAM-8218] WIP, PulsarIO, looking forward for initial feedback

2022-01-27 Thread GitBox
MarcoRob commented on pull request #15572: URL: https://github.com/apache/beam/pull/15572#issuecomment-1023547897 Hi folks, I am closing this PR since I finished the first version of PulsarIO. Here you can find the new PR created with the latest changes commited, [PR-16634](https://

[GitHub] [beam] MarcoRob closed pull request #15572: [BEAM-8218] WIP, PulsarIO, looking forward for initial feedback

2022-01-27 Thread GitBox
MarcoRob closed pull request #15572: URL: https://github.com/apache/beam/pull/15572 -- 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 the specific comment. To unsubscribe, e-mail: github-unsubsc

[GitHub] [beam] ibzib commented on pull request #16513: [BEAM-12976] Implement Java projection pushdown optimizer.

2022-01-27 Thread GitBox
ibzib commented on pull request #16513: URL: https://github.com/apache/beam/pull/16513#issuecomment-1023552732 Run Java PreCommit -- 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 the specific com

[GitHub] [beam] MarcoRob edited a comment on pull request #15572: [BEAM-8218] WIP, PulsarIO, looking forward for initial feedback

2022-01-27 Thread GitBox
MarcoRob edited a comment on pull request #15572: URL: https://github.com/apache/beam/pull/15572#issuecomment-1023547897 Hi folks, I am closing this PR since I finished the first version of PulsarIO and have a cleaner PR with commits. Here you can find the new PR created with the lat

[GitHub] [beam] ibzib commented on pull request #16513: [BEAM-12976] Implement Java projection pushdown optimizer.

2022-01-27 Thread GitBox
ibzib commented on pull request #16513: URL: https://github.com/apache/beam/pull/16513#issuecomment-1023560336 Run Java PreCommit -- 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 the specific com

[GitHub] [beam] codecov[bot] edited a comment on pull request #14355: [WIP][BEAM-11976] Bump org.apache.kafka:kafka_2.12 to 2.7.0

2022-01-27 Thread GitBox
codecov[bot] edited a comment on pull request #14355: URL: https://github.com/apache/beam/pull/14355#issuecomment-809510122 # [Codecov](https://codecov.io/gh/apache/beam/pull/14355?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+A

[GitHub] [beam] dannymartinm commented on a change in pull request #16511: [BEAM-12812] - Run Github Actions on GCP workers in the apache-beam-testing project

2022-01-27 Thread GitBox
dannymartinm commented on a change in pull request #16511: URL: https://github.com/apache/beam/pull/16511#discussion_r793930463 ## File path: .github/dockerized-gh-actions-runners/self-hosted-linux/docker/entrypoint.sh ## @@ -0,0 +1,46 @@ +#!/usr/bin/env bash + +# Licensed to

[GitHub] [beam] ihji opened a new pull request #16635: Fix google3 import error due to missing Python gradle build file

2022-01-27 Thread GitBox
ihji opened a new pull request #16635: URL: https://github.com/apache/beam/pull/16635 **Please** add a meaningful description for your change here Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly a

[GitHub] [beam] ibzib commented on pull request #16513: [BEAM-12976] Implement Java projection pushdown optimizer.

2022-01-27 Thread GitBox
ibzib commented on pull request #16513: URL: https://github.com/apache/beam/pull/16513#issuecomment-1023572713 Run Java PreCommit -- 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 the specific com

[GitHub] [beam] aaltay commented on pull request #14350: [BEAM-9613] fix handling of BQ BOOLEAN, FLOAT, REPEATED

2022-01-27 Thread GitBox
aaltay commented on pull request #14350: URL: https://github.com/apache/beam/pull/14350#issuecomment-1023573764 > @aaltay I haven't had a chance to do further work on this and I'm not sure when I will get the chance, but I definitely think this needs to be addressed because it's definitely

[GitHub] [beam] codecov[bot] edited a comment on pull request #14355: [WIP][BEAM-11976] Bump org.apache.kafka:kafka_2.12 to 2.7.0

2022-01-27 Thread GitBox
codecov[bot] edited a comment on pull request #14355: URL: https://github.com/apache/beam/pull/14355#issuecomment-809510122 -- 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 the specific comment.

[GitHub] [beam] codecov[bot] edited a comment on pull request #16632: [DO NOT MERGE] Run Python RC Validation Tests

2022-01-27 Thread GitBox
codecov[bot] edited a comment on pull request #16632: URL: https://github.com/apache/beam/pull/16632#issuecomment-1023518593 # [Codecov](https://codecov.io/gh/apache/beam/pull/16632?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+

[GitHub] [beam] kennknowles commented on pull request #16604: BEAM-13714 Get rid of unnecessary logs for BigQuery streaming writes

2022-01-27 Thread GitBox
kennknowles commented on pull request #16604: URL: https://github.com/apache/beam/pull/16604#issuecomment-1023578891 R: @scwhittle You agree that these logs are superfluous and probably good to remove them to reduce volume? -- This is an automated message from the Apache Git Serv

[GitHub] [beam] codecov[bot] edited a comment on pull request #16632: [DO NOT MERGE] Run Python RC Validation Tests

2022-01-27 Thread GitBox
codecov[bot] edited a comment on pull request #16632: URL: https://github.com/apache/beam/pull/16632#issuecomment-1023518593 # [Codecov](https://codecov.io/gh/apache/beam/pull/16632?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+

[GitHub] [beam] scwhittle commented on pull request #16604: BEAM-13714 Get rid of unnecessary logs for BigQuery streaming writes

2022-01-27 Thread GitBox
scwhittle commented on pull request #16604: URL: https://github.com/apache/beam/pull/16604#issuecomment-1023582606 Yes it is definitely spammy, it could perhaps be kept as a debug log if possibly of interest in debugging how effective batching is. But perhaps some metrics or periodic loggi

[GitHub] [beam] codecov[bot] edited a comment on pull request #16632: [DO NOT MERGE] Run Python RC Validation Tests

2022-01-27 Thread GitBox
codecov[bot] edited a comment on pull request #16632: URL: https://github.com/apache/beam/pull/16632#issuecomment-1023518593 # [Codecov](https://codecov.io/gh/apache/beam/pull/16632?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+

[GitHub] [beam] codecov[bot] edited a comment on pull request #16632: [DO NOT MERGE] Run Python RC Validation Tests

2022-01-27 Thread GitBox
codecov[bot] edited a comment on pull request #16632: URL: https://github.com/apache/beam/pull/16632#issuecomment-1023518593 # [Codecov](https://codecov.io/gh/apache/beam/pull/16632?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+

[GitHub] [beam] codecov[bot] edited a comment on pull request #14355: [WIP][BEAM-11976] Bump org.apache.kafka:kafka_2.12 to 2.7.0

2022-01-27 Thread GitBox
codecov[bot] edited a comment on pull request #14355: URL: https://github.com/apache/beam/pull/14355#issuecomment-809510122 # [Codecov](https://codecov.io/gh/apache/beam/pull/14355?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+A

[GitHub] [beam] codecov[bot] edited a comment on pull request #14355: [WIP][BEAM-11976] Bump org.apache.kafka:kafka_2.12 to 2.7.0

2022-01-27 Thread GitBox
codecov[bot] edited a comment on pull request #14355: URL: https://github.com/apache/beam/pull/14355#issuecomment-809510122 # [Codecov](https://codecov.io/gh/apache/beam/pull/14355?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+A

[GitHub] [beam] codecov[bot] edited a comment on pull request #16632: [DO NOT MERGE] Run Python RC Validation Tests

2022-01-27 Thread GitBox
codecov[bot] edited a comment on pull request #16632: URL: https://github.com/apache/beam/pull/16632#issuecomment-1023518593 # [Codecov](https://codecov.io/gh/apache/beam/pull/16632?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+

[GitHub] [beam] kennknowles merged pull request #16604: [BEAM-13714] Get rid of unnecessary logs for BigQuery streaming writes

2022-01-27 Thread GitBox
kennknowles merged pull request #16604: URL: https://github.com/apache/beam/pull/16604 -- 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 the specific comment. To unsubscribe, e-mail: github-unsu

[GitHub] [beam] codecov[bot] edited a comment on pull request #16632: [DO NOT MERGE] Run Python RC Validation Tests

2022-01-27 Thread GitBox
codecov[bot] edited a comment on pull request #16632: URL: https://github.com/apache/beam/pull/16632#issuecomment-1023518593 # [Codecov](https://codecov.io/gh/apache/beam/pull/16632?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+

[GitHub] [beam] codecov[bot] commented on pull request #16633: [BEAM-4032]Support staging binary distributions of dependency packages

2022-01-27 Thread GitBox
codecov[bot] commented on pull request #16633: URL: https://github.com/apache/beam/pull/16633#issuecomment-1023597482 # [Codecov](https://codecov.io/gh/apache/beam/pull/16633?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+

[GitHub] [beam] codecov[bot] edited a comment on pull request #16629: [BEAM-13757] adds pane observation in DoFn

2022-01-27 Thread GitBox
codecov[bot] edited a comment on pull request #16629: URL: https://github.com/apache/beam/pull/16629#issuecomment-1023486045 # [Codecov](https://codecov.io/gh/apache/beam/pull/16629?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+

[GitHub] [beam] codecov[bot] edited a comment on pull request #16633: [BEAM-4032]Support staging binary distributions of dependency packages

2022-01-27 Thread GitBox
codecov[bot] edited a comment on pull request #16633: URL: https://github.com/apache/beam/pull/16633#issuecomment-1023597482 # [Codecov](https://codecov.io/gh/apache/beam/pull/16633?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+

[GitHub] [beam] codecov[bot] edited a comment on pull request #14355: [WIP][BEAM-11976] Bump org.apache.kafka:kafka_2.12 to 2.7.0

2022-01-27 Thread GitBox
codecov[bot] edited a comment on pull request #14355: URL: https://github.com/apache/beam/pull/14355#issuecomment-809510122 # [Codecov](https://codecov.io/gh/apache/beam/pull/14355?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+A

[GitHub] [beam] codecov[bot] edited a comment on pull request #16629: [BEAM-13757] adds pane observation in DoFn

2022-01-27 Thread GitBox
codecov[bot] edited a comment on pull request #16629: URL: https://github.com/apache/beam/pull/16629#issuecomment-1023486045 # [Codecov](https://codecov.io/gh/apache/beam/pull/16629?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+

[GitHub] [beam] codecov[bot] edited a comment on pull request #16632: [DO NOT MERGE] Run Python RC Validation Tests

2022-01-27 Thread GitBox
codecov[bot] edited a comment on pull request #16632: URL: https://github.com/apache/beam/pull/16632#issuecomment-1023518593 # [Codecov](https://codecov.io/gh/apache/beam/pull/16632?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+

[GitHub] [beam] AnandInguva opened a new pull request #16636: Fix bug in the main_session unit test

2022-01-27 Thread GitBox
AnandInguva opened a new pull request #16636: URL: https://github.com/apache/beam/pull/16636 **Please** add a meaningful description for your change here Thank you for your contribution! Follow this checklist to help us incorporate your contribution qu

[GitHub] [beam] codecov[bot] edited a comment on pull request #16632: [DO NOT MERGE] Run Python RC Validation Tests

2022-01-27 Thread GitBox
codecov[bot] edited a comment on pull request #16632: URL: https://github.com/apache/beam/pull/16632#issuecomment-1023518593 # [Codecov](https://codecov.io/gh/apache/beam/pull/16632?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+

[GitHub] [beam] codecov[bot] edited a comment on pull request #16633: [BEAM-4032]Support staging binary distributions of dependency packages

2022-01-27 Thread GitBox
codecov[bot] edited a comment on pull request #16633: URL: https://github.com/apache/beam/pull/16633#issuecomment-1023597482 # [Codecov](https://codecov.io/gh/apache/beam/pull/16633?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+

[GitHub] [beam] codecov[bot] edited a comment on pull request #16633: [BEAM-4032]Support staging binary distributions of dependency packages

2022-01-27 Thread GitBox
codecov[bot] edited a comment on pull request #16633: URL: https://github.com/apache/beam/pull/16633#issuecomment-1023597482 # [Codecov](https://codecov.io/gh/apache/beam/pull/16633?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+

  1   2   3   >