[GitHub] [beam] yyy1000 commented on pull request #25203: Fix interface{} in iter& emit type of DoFn in Go

2023-02-15 Thread via GitHub
yyy1000 commented on PR #25203: URL: https://github.com/apache/beam/pull/25203#issuecomment-1432607554 So far it's close to finished.😩 It seems that I made a lot of mistakes during the process, and I will do better next time. Thank you for your patient guidance. :) -- This is an

[GitHub] [beam] yyy1000 commented on a diff in pull request #25203: Fix interface{} in iter& emit type of DoFn in Go

2023-02-15 Thread via GitHub
yyy1000 commented on code in PR #25203: URL: https://github.com/apache/beam/pull/25203#discussion_r1108068105 ## sdks/go/pkg/beam/core/funcx/sideinput_test.go: ## @@ -39,6 +39,7 @@ func TestIsIter(t *testing.T) { {func(*typex.EventTime, *int, *string) bool { retu

[GitHub] [beam] lostluck commented on a diff in pull request #25203: Fix interface{} in iter& emit type of DoFn in Go

2023-02-15 Thread via GitHub
lostluck commented on code in PR #25203: URL: https://github.com/apache/beam/pull/25203#discussion_r1108036547 ## sdks/go/pkg/beam/core/funcx/sideinput_test.go: ## @@ -39,6 +39,7 @@ func TestIsIter(t *testing.T) { {func(*typex.EventTime, *int, *string) bool { ret

[GitHub] [beam] codecov[bot] commented on pull request #25504: Update pyhamcrest requirement from !=1.10.0,<2.0.0,>=1.9 to >=1.9,!=1.10.0,<3.0.0 in /sdks/python

2023-02-15 Thread via GitHub
codecov[bot] commented on PR #25504: URL: https://github.com/apache/beam/pull/25504#issuecomment-1432532279 # [Codecov](https://codecov.io/gh/apache/beam/pull/25504?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Fo

[GitHub] [beam] codecov[bot] commented on pull request #25503: Update mock requirement from <3.0.0,>=1.0.1 to >=1.0.1,<6.0.0 in /sdks/python

2023-02-15 Thread via GitHub
codecov[bot] commented on PR #25503: URL: https://github.com/apache/beam/pull/25503#issuecomment-1432530984 # [Codecov](https://codecov.io/gh/apache/beam/pull/25503?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Fo

[GitHub] [beam] dependabot[bot] opened a new pull request, #25505: Bump github.com/aws/aws-sdk-go-v2/config from 1.18.11 to 1.18.13 in /sdks

2023-02-15 Thread via GitHub
dependabot[bot] opened a new pull request, #25505: URL: https://github.com/apache/beam/pull/25505 Bumps [github.com/aws/aws-sdk-go-v2/config](https://github.com/aws/aws-sdk-go-v2) from 1.18.11 to 1.18.13. Changelog Sourced from https://github.com/aws/aws-sdk-go-v2/blob/main/CHANGE

[GitHub] [beam] dependabot[bot] closed pull request #25324: Bump github.com/aws/aws-sdk-go-v2/config from 1.18.11 to 1.18.12 in /sdks

2023-02-15 Thread via GitHub
dependabot[bot] closed pull request #25324: Bump github.com/aws/aws-sdk-go-v2/config from 1.18.11 to 1.18.12 in /sdks URL: https://github.com/apache/beam/pull/25324 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [beam] dependabot[bot] opened a new pull request, #25503: Update mock requirement from <3.0.0,>=1.0.1 to >=1.0.1,<6.0.0 in /sdks/python

2023-02-15 Thread via GitHub
dependabot[bot] opened a new pull request, #25503: URL: https://github.com/apache/beam/pull/25503 Updates the requirements on [mock](https://github.com/testing-cabal/mock) to permit the latest version. Changelog Sourced from https://github.com/testing-cabal/mock/blob/master/CHANGEL

[GitHub] [beam] dependabot[bot] opened a new pull request, #25504: Update pyhamcrest requirement from !=1.10.0,<2.0.0,>=1.9 to >=1.9,!=1.10.0,<3.0.0 in /sdks/python

2023-02-15 Thread via GitHub
dependabot[bot] opened a new pull request, #25504: URL: https://github.com/apache/beam/pull/25504 Updates the requirements on [pyhamcrest](https://github.com/hamcrest/PyHamcrest) to permit the latest version. Changelog Sourced from https://github.com/hamcrest/PyHamcrest/blob/main/

[GitHub] [beam] Malarg commented on a diff in pull request #25397: Issue25329 fix cors issue when no internet

2023-02-15 Thread via GitHub
Malarg commented on code in PR #25397: URL: https://github.com/apache/beam/pull/25397#discussion_r1107993350 ## playground/frontend/playground_components/assets/translations/en.yaml: ## @@ -17,12 +17,16 @@ errors: error: 'Error' + failedParseOptions: "Failed to parse pipe

[GitHub] [beam] lostluck commented on pull request #25437: [Go SDK]: Allow SDF methods to have context param and error return value

2023-02-15 Thread via GitHub
lostluck commented on PR #25437: URL: https://github.com/apache/beam/pull/25437#issuecomment-1432382838 Work got away from me, preventing me from spending time on this. First thing I do tomorrow though. -- This is an automated message from the Apache Git Service. To respond to the message

[GitHub] [beam-starter-java] dependabot[bot] opened a new pull request, #48: Bump beam-sdks-java-core from 2.44.0 to 2.45.0

2023-02-15 Thread via GitHub
dependabot[bot] opened a new pull request, #48: URL: https://github.com/apache/beam-starter-java/pull/48 Bumps [beam-sdks-java-core](https://github.com/apache/beam) from 2.44.0 to 2.45.0. Release notes Sourced from https://github.com/apache/beam/releases";>beam-sdks-java-core's re

[GitHub] [beam-starter-java] dependabot[bot] opened a new pull request, #47: Bump org.apache.beam:beam-runners-direct-java from 2.44.0 to 2.45.0

2023-02-15 Thread via GitHub
dependabot[bot] opened a new pull request, #47: URL: https://github.com/apache/beam-starter-java/pull/47 Bumps [org.apache.beam:beam-runners-direct-java](https://github.com/apache/beam) from 2.44.0 to 2.45.0. Release notes Sourced from https://github.com/apache/beam/releases";>org

[GitHub] [beam-starter-java] dependabot[bot] opened a new pull request, #45: Bump org.apache.beam:beam-sdks-java-core from 2.44.0 to 2.45.0

2023-02-15 Thread via GitHub
dependabot[bot] opened a new pull request, #45: URL: https://github.com/apache/beam-starter-java/pull/45 Bumps [org.apache.beam:beam-sdks-java-core](https://github.com/apache/beam) from 2.44.0 to 2.45.0. Release notes Sourced from https://github.com/apache/beam/releases";>org.apach

[GitHub] [beam-starter-java] dependabot[bot] opened a new pull request, #46: Bump beam-runners-direct-java from 2.44.0 to 2.45.0

2023-02-15 Thread via GitHub
dependabot[bot] opened a new pull request, #46: URL: https://github.com/apache/beam-starter-java/pull/46 Bumps [beam-runners-direct-java](https://github.com/apache/beam) from 2.44.0 to 2.45.0. Release notes Sourced from https://github.com/apache/beam/releases";>beam-runners-direct-

[GitHub] [beam] pof-declaneaston commented on issue #25502: [Bug]: Zero Backlog for DataFlow Python Kafka Pipeline

2023-02-15 Thread via GitHub
pof-declaneaston commented on issue #25502: URL: https://github.com/apache/beam/issues/25502#issuecomment-1432334863 I tried running the pipeline with num_workers = 3, which worked well for a little while but eventually the runner scaled it back down to 1 worker and the system is unable to

[GitHub] [beam] CAM-Gerlach commented on issue #18844: Non-working Python 3 versions are installable on PyPI

2023-02-15 Thread via GitHub
CAM-Gerlach commented on issue #18844: URL: https://github.com/apache/beam/issues/18844#issuecomment-1432327499 I figure it's harmless, but just in case—after commenting on this issue I got two GH Actions failure notifications: * https://github.com/apache/beam/actions/runs/4169541700

[GitHub] [beam] yyy1000 commented on a diff in pull request #25203: Fix interface{} in iter& emit type of DoFn in Go

2023-02-15 Thread via GitHub
yyy1000 commented on code in PR #25203: URL: https://github.com/apache/beam/pull/25203#discussion_r1107909513 ## sdks/go/pkg/beam/core/funcx/sideinput_test.go: ## @@ -39,6 +39,7 @@ func TestIsIter(t *testing.T) { {func(*typex.EventTime, *int, *string) bool { retu

[GitHub] [beam] alxp1982 commented on a diff in pull request #25257: [Tour of Beam] Learning content for "Triggers" module

2023-02-15 Thread via GitHub
alxp1982 commented on code in PR #25257: URL: https://github.com/apache/beam/pull/25257#discussion_r1107859379 ## learning/tour-of-beam/learning-content/triggers/concept/description.md: ## @@ -0,0 +1,147 @@ + + +### Trigger Review Comment: ### Triggers ## learning

[GitHub] [beam] pof-declaneaston opened a new issue, #25502: [Bug]: Zero Backlog for DataFlow Python Kafka Pipeline

2023-02-15 Thread via GitHub
pof-declaneaston opened a new issue, #25502: URL: https://github.com/apache/beam/issues/25502 ### What happened? Hello, I have built a DataFlow Python pipeline which I am using to consume from and produce to Kafka. When I run my pipeline I notice that the backlog metric in the

[GitHub] [beam] dependabot[bot] commented on pull request #25440: Bump grpcio from 1.49.1 to 1.51.1 in /sdks/python

2023-02-15 Thread via GitHub
dependabot[bot] commented on PR #25440: URL: https://github.com/apache/beam/pull/25440#issuecomment-1432302964 OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor version, let

[GitHub] [beam] tvalentyn closed pull request #25440: Bump grpcio from 1.49.1 to 1.51.1 in /sdks/python

2023-02-15 Thread via GitHub
tvalentyn closed pull request #25440: Bump grpcio from 1.49.1 to 1.51.1 in /sdks/python URL: https://github.com/apache/beam/pull/25440 -- 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] commented on pull request #25501: Bump protobuf from 3.19.4 to 3.19.5 in /sdks/python/container/py310

2023-02-15 Thread via GitHub
codecov[bot] commented on PR #25501: URL: https://github.com/apache/beam/pull/25501#issuecomment-1432264049 # [Codecov](https://codecov.io/gh/apache/beam/pull/25501?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Fo

[GitHub] [beam] dependabot[bot] closed pull request #25378: Bump cryptography from 39.0.0 to 39.0.1 in /sdks/python/container/py310

2023-02-15 Thread via GitHub
dependabot[bot] closed pull request #25378: Bump cryptography from 39.0.0 to 39.0.1 in /sdks/python/container/py310 URL: https://github.com/apache/beam/pull/25378 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL ab

[GitHub] [beam] dependabot[bot] commented on pull request #25378: Bump cryptography from 39.0.0 to 39.0.1 in /sdks/python/container/py310

2023-02-15 Thread via GitHub
dependabot[bot] commented on PR #25378: URL: https://github.com/apache/beam/pull/25378#issuecomment-1432236948 Looks like cryptography is up-to-date now, so this is no longer needed. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Gi

[GitHub] [beam] dependabot[bot] closed pull request #25376: Bump cryptography from 39.0.0 to 39.0.1 in /sdks/python/container/py37

2023-02-15 Thread via GitHub
dependabot[bot] closed pull request #25376: Bump cryptography from 39.0.0 to 39.0.1 in /sdks/python/container/py37 URL: https://github.com/apache/beam/pull/25376 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abo

[GitHub] [beam] dependabot[bot] opened a new pull request, #25501: Bump protobuf from 3.19.4 to 3.19.5 in /sdks/python/container/py310

2023-02-15 Thread via GitHub
dependabot[bot] opened a new pull request, #25501: URL: https://github.com/apache/beam/pull/25501 Bumps [protobuf](https://github.com/protocolbuffers/protobuf) from 3.19.4 to 3.19.5. Release notes Sourced from https://github.com/protocolbuffers/protobuf/releases";>protobuf's relea

[GitHub] [beam] dependabot[bot] commented on pull request #25376: Bump cryptography from 39.0.0 to 39.0.1 in /sdks/python/container/py37

2023-02-15 Thread via GitHub
dependabot[bot] commented on PR #25376: URL: https://github.com/apache/beam/pull/25376#issuecomment-1432236937 Looks like cryptography is up-to-date now, so this is no longer needed. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Gi

[GitHub] [beam] tvalentyn merged pull request #25475: Update python container images

2023-02-15 Thread via GitHub
tvalentyn merged PR #25475: URL: https://github.com/apache/beam/pull/25475 -- 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...@beam.apach

[GitHub] [beam] tvalentyn commented on pull request #23876: Fix beam.Row.__eq__ for rows with trailing columns

2023-02-15 Thread via GitHub
tvalentyn commented on PR #23876: URL: https://github.com/apache/beam/pull/23876#issuecomment-1432235610 R: @damccorm -- 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 un

[GitHub] [beam] mutianf commented on pull request #24015: [Bigtable] Migrate BigtableIO to use the veneer client under the hood

2023-02-15 Thread via GitHub
mutianf commented on PR #24015: URL: https://github.com/apache/beam/pull/24015#issuecomment-1432223299 Sorry @Abacn, we decided to refactor the code so it's taking longer. I'll mark it as draft for now so you won't keep getting pinged. :( -- This is an automated message from the Apache G

[GitHub] [beam] vachan-shetty commented on pull request #25392: Read API Source v2

2023-02-15 Thread via GitHub
vachan-shetty commented on PR #25392: URL: https://github.com/apache/beam/pull/25392#issuecomment-1432182120 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 comme

[GitHub] [beam] alxp1982 commented on a diff in pull request #25258: [Tour of Beam] Learning content for "Windowing" module

2023-02-15 Thread via GitHub
alxp1982 commented on code in PR #25258: URL: https://github.com/apache/beam/pull/25258#discussion_r1107767973 ## learning/tour-of-beam/learning-content/windowing/motivating-challenge/description.md: ## @@ -0,0 +1,25 @@ + + +### Common Transforms motivating challenge + +You are

[GitHub] [beam] johnjcasey merged pull request #25500: Update documents for 2.45

2023-02-15 Thread via GitHub
johnjcasey merged PR #25500: URL: https://github.com/apache/beam/pull/25500 -- 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...@beam.apac

[GitHub] [beam] johnjcasey opened a new pull request, #25500: Update documents for 2.45

2023-02-15 Thread via GitHub
johnjcasey opened a new pull request, #25500: URL: https://github.com/apache/beam/pull/25500 **Please** add a meaningful description for your change here Thank you for your contribution! Follow this checklist to help us incorporate your contribution qui

[GitHub] [beam] codecov[bot] commented on pull request #25496: [Python] Support loading models with saved weights

2023-02-15 Thread via GitHub
codecov[bot] commented on PR #25496: URL: https://github.com/apache/beam/pull/25496#issuecomment-1432067599 # [Codecov](https://codecov.io/gh/apache/beam/pull/25496?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Fo

[GitHub] [beam] johnjcasey merged pull request #25407: Update documents for 2.45

2023-02-15 Thread via GitHub
johnjcasey merged PR #25407: URL: https://github.com/apache/beam/pull/25407 -- 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...@beam.apac

[GitHub] [beam] Edinsonmyosa opened a new issue, #25499: [Bug]: The Write function is no longer available in datastoreio

2023-02-15 Thread via GitHub
Edinsonmyosa opened a new issue, #25499: URL: https://github.com/apache/beam/issues/25499 ### What happened? In some information I have seen the function Write as the way to put back updated information in GCP datastore. I'm trying to do that but I can only find a Read() function.

[GitHub] [beam] tonytanger commented on pull request #25470: Update GCP cloud libraries BOM to 26.8.0

2023-02-15 Thread via GitHub
tonytanger commented on PR #25470: URL: https://github.com/apache/beam/pull/25470#issuecomment-1432052557 Run Java_PVR_Flink_Batch 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 sp

[GitHub] [beam] lostluck commented on a diff in pull request #25203: Fix interface{} in iter& emit type of DoFn in Go

2023-02-15 Thread via GitHub
lostluck commented on code in PR #25203: URL: https://github.com/apache/beam/pull/25203#discussion_r1107736060 ## sdks/go/pkg/beam/core/funcx/sideinput_test.go: ## @@ -39,6 +39,7 @@ func TestIsIter(t *testing.T) { {func(*typex.EventTime, *int, *string) bool { ret

[GitHub] [beam] lostluck commented on a diff in pull request #25476: [#24789][prism] internal/coders.go and tests

2023-02-15 Thread via GitHub
lostluck commented on code in PR #25476: URL: https://github.com/apache/beam/pull/25476#discussion_r1107733619 ## sdks/go/pkg/beam/runners/prism/internal/coders.go: ## @@ -0,0 +1,243 @@ +// Licensed to the Apache Software Foundation (ASF) under one or more +// contributor licens

[GitHub] [beam] lostluck merged pull request #25476: [#24789][prism] internal/coders.go and tests

2023-02-15 Thread via GitHub
lostluck merged PR #25476: URL: https://github.com/apache/beam/pull/25476 -- 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...@beam.apache

[GitHub] [beam] Abacn commented on a diff in pull request #25400: Update BigQueryIO documentation with details on how to override the query execution project

2023-02-15 Thread via GitHub
Abacn commented on code in PR #25400: URL: https://github.com/apache/beam/pull/25400#discussion_r1107726259 ## website/www/site/content/en/documentation/io/built-in/google-bigquery.md: ## @@ -323,6 +323,11 @@ in the following example: {{< code_sample "sdks/python/apache_beam/ex

[GitHub] [beam] lostluck commented on pull request #25497: [#24789][prism] internal/jobservices metrics + tests

2023-02-15 Thread via GitHub
lostluck commented on PR #25497: URL: https://github.com/apache/beam/pull/25497#issuecomment-1432042397 R: @jrmccluskey -- 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

[GitHub] [beam] lostluck commented on pull request #25497: [#24789][prism] internal/jobservices metrics + tests

2023-02-15 Thread via GitHub
lostluck commented on PR #25497: URL: https://github.com/apache/beam/pull/25497#issuecomment-1432041848 Run GoPortable 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 comme

[GitHub] [beam-starter-kotlin] dependabot[bot] commented on pull request #15: Bump beamVersion from 2.42.0 to 2.44.0

2023-02-15 Thread via GitHub
dependabot[bot] commented on PR #15: URL: https://github.com/apache/beam-starter-kotlin/pull/15#issuecomment-1432031250 Superseded by #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 the spe

[GitHub] [beam-starter-kotlin] dependabot[bot] closed pull request #15: Bump beamVersion from 2.42.0 to 2.44.0

2023-02-15 Thread via GitHub
dependabot[bot] closed pull request #15: Bump beamVersion from 2.42.0 to 2.44.0 URL: https://github.com/apache/beam-starter-kotlin/pull/15 -- 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-starter-kotlin] dependabot[bot] opened a new pull request, #17: Bump beamVersion from 2.42.0 to 2.45.0

2023-02-15 Thread via GitHub
dependabot[bot] opened a new pull request, #17: URL: https://github.com/apache/beam-starter-kotlin/pull/17 Bumps `beamVersion` from 2.42.0 to 2.45.0. Updates `org.apache.beam:beam-sdks-java-core` from 2.42.0 to 2.45.0 Release notes Sourced from https://github.com/apache/beam/rele

[GitHub] [beam-starter-kotlin] dependabot[bot] commented on pull request #17: Bump beamVersion from 2.42.0 to 2.45.0

2023-02-15 Thread via GitHub
dependabot[bot] commented on PR #17: URL: https://github.com/apache/beam-starter-kotlin/pull/17#issuecomment-1432031224 Dependabot tried to add `@tosun-si`, `@lukecwik`, `@kennknowles`, `@robertwb` and `@kileys` as reviewers to this PR, but received the following error from GitHub:

[GitHub] [beam] bvolpato opened a new issue, #25498: [Bug]: Failure when reading non UTF-8 requirements.txt file

2023-02-15 Thread via GitHub
bvolpato opened a new issue, #25498: URL: https://github.com/apache/beam/issues/25498 ### What happened? This error caused some customer friction and took quite a bit to figure out what was going wrong: ``` INFO 2023-02-06T20:16:45.090490Z File "/usr/local/lib/python3.9/sit

[GitHub] [beam] lostluck opened a new pull request, #25497: [#24789][prism] internal/jobservices metrics + tests

2023-02-15 Thread via GitHub
lostluck opened a new pull request, #25497: URL: https://github.com/apache/beam/pull/25497 Adds metrics store handling and initial unit tests. For Prism's initial implementation. See https://github.com/apache/beam/issues/24789. Thank you fo

[GitHub] [beam] tvalentyn commented on pull request #25488: [Python] fix pipeline parsing with single dash

2023-02-15 Thread via GitHub
tvalentyn commented on PR #25488: URL: https://github.com/apache/beam/pull/25488#issuecomment-1432016307 (and take a look at the failures) -- 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 spec

[GitHub] [beam] riteshghorse opened a new pull request, #25496: [Python] Support loading models with saved weights

2023-02-15 Thread via GitHub
riteshghorse opened a new pull request, #25496: URL: https://github.com/apache/beam/pull/25496 **Please** add a meaningful description for your change here Thank you for your contribution! Follow this checklist to help us incorporate your contribution q

[GitHub] [beam] tvalentyn commented on pull request #25488: [Python] fix pipeline parsing with single dash

2023-02-15 Thread via GitHub
tvalentyn commented on PR #25488: URL: https://github.com/apache/beam/pull/25488#issuecomment-1432015444 Thanks, can you please add a unit tests? -- 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] dependabot[bot] commented on pull request #25493: Bump werkzeug from 2.2.2 to 2.2.3 in /sdks/python/container/py39

2023-02-15 Thread via GitHub
dependabot[bot] commented on PR #25493: URL: https://github.com/apache/beam/pull/25493#issuecomment-1432013729 OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor version, let

[GitHub] [beam] damccorm closed pull request #25493: Bump werkzeug from 2.2.2 to 2.2.3 in /sdks/python/container/py39

2023-02-15 Thread via GitHub
damccorm closed pull request #25493: Bump werkzeug from 2.2.2 to 2.2.3 in /sdks/python/container/py39 URL: https://github.com/apache/beam/pull/25493 -- 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] damccorm closed pull request #25492: Bump werkzeug from 2.2.2 to 2.2.3 in /sdks/python/container/py310

2023-02-15 Thread via GitHub
damccorm closed pull request #25492: Bump werkzeug from 2.2.2 to 2.2.3 in /sdks/python/container/py310 URL: https://github.com/apache/beam/pull/25492 -- 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

[GitHub] [beam] dependabot[bot] commented on pull request #25492: Bump werkzeug from 2.2.2 to 2.2.3 in /sdks/python/container/py310

2023-02-15 Thread via GitHub
dependabot[bot] commented on PR #25492: URL: https://github.com/apache/beam/pull/25492#issuecomment-1432013779 OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor version, let

[GitHub] [beam] dependabot[bot] commented on pull request #25495: Bump werkzeug from 2.2.2 to 2.2.3 in /sdks/python/container/py37

2023-02-15 Thread via GitHub
dependabot[bot] commented on PR #25495: URL: https://github.com/apache/beam/pull/25495#issuecomment-1432013415 OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor version, let

[GitHub] [beam] damccorm closed pull request #25494: Bump werkzeug from 2.2.2 to 2.2.3 in /sdks/python/container/py38

2023-02-15 Thread via GitHub
damccorm closed pull request #25494: Bump werkzeug from 2.2.2 to 2.2.3 in /sdks/python/container/py38 URL: https://github.com/apache/beam/pull/25494 -- 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] dependabot[bot] commented on pull request #25494: Bump werkzeug from 2.2.2 to 2.2.3 in /sdks/python/container/py38

2023-02-15 Thread via GitHub
dependabot[bot] commented on PR #25494: URL: https://github.com/apache/beam/pull/25494#issuecomment-1432013667 OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor version, let

[GitHub] [beam] damccorm closed pull request #25495: Bump werkzeug from 2.2.2 to 2.2.3 in /sdks/python/container/py37

2023-02-15 Thread via GitHub
damccorm closed pull request #25495: Bump werkzeug from 2.2.2 to 2.2.3 in /sdks/python/container/py37 URL: https://github.com/apache/beam/pull/25495 -- 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] jrmccluskey commented on a diff in pull request #25476: [#24789][prism] internal/coders.go and tests

2023-02-15 Thread via GitHub
jrmccluskey commented on code in PR #25476: URL: https://github.com/apache/beam/pull/25476#discussion_r1107690382 ## sdks/go/pkg/beam/runners/prism/internal/coders.go: ## @@ -0,0 +1,243 @@ +// Licensed to the Apache Software Foundation (ASF) under one or more +// contributor lic

[GitHub] [beam] codecov[bot] commented on pull request #25492: Bump werkzeug from 2.2.2 to 2.2.3 in /sdks/python/container/py310

2023-02-15 Thread via GitHub
codecov[bot] commented on PR #25492: URL: https://github.com/apache/beam/pull/25492#issuecomment-1432007911 # [Codecov](https://codecov.io/gh/apache/beam/pull/25492?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Fo

[GitHub] [beam] codecov[bot] commented on pull request #25494: Bump werkzeug from 2.2.2 to 2.2.3 in /sdks/python/container/py38

2023-02-15 Thread via GitHub
codecov[bot] commented on PR #25494: URL: https://github.com/apache/beam/pull/25494#issuecomment-1432007210 # [Codecov](https://codecov.io/gh/apache/beam/pull/25494?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Fo

[GitHub] [beam] codecov[bot] commented on pull request #25495: Bump werkzeug from 2.2.2 to 2.2.3 in /sdks/python/container/py37

2023-02-15 Thread via GitHub
codecov[bot] commented on PR #25495: URL: https://github.com/apache/beam/pull/25495#issuecomment-1432005357 # [Codecov](https://codecov.io/gh/apache/beam/pull/25495?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Fo

[GitHub] [beam] dependabot[bot] opened a new pull request, #25493: Bump werkzeug from 2.2.2 to 2.2.3 in /sdks/python/container/py39

2023-02-15 Thread via GitHub
dependabot[bot] opened a new pull request, #25493: URL: https://github.com/apache/beam/pull/25493 Bumps [werkzeug](https://github.com/pallets/werkzeug) from 2.2.2 to 2.2.3. Release notes Sourced from https://github.com/pallets/werkzeug/releases";>werkzeug's releases. 2.2.3

[GitHub] [beam] dependabot[bot] opened a new pull request, #25494: Bump werkzeug from 2.2.2 to 2.2.3 in /sdks/python/container/py38

2023-02-15 Thread via GitHub
dependabot[bot] opened a new pull request, #25494: URL: https://github.com/apache/beam/pull/25494 Bumps [werkzeug](https://github.com/pallets/werkzeug) from 2.2.2 to 2.2.3. Release notes Sourced from https://github.com/pallets/werkzeug/releases";>werkzeug's releases. 2.2.3

[GitHub] [beam] dependabot[bot] opened a new pull request, #25495: Bump werkzeug from 2.2.2 to 2.2.3 in /sdks/python/container/py37

2023-02-15 Thread via GitHub
dependabot[bot] opened a new pull request, #25495: URL: https://github.com/apache/beam/pull/25495 Bumps [werkzeug](https://github.com/pallets/werkzeug) from 2.2.2 to 2.2.3. Release notes Sourced from https://github.com/pallets/werkzeug/releases";>werkzeug's releases. 2.2.3

[GitHub] [beam] dependabot[bot] opened a new pull request, #25492: Bump werkzeug from 2.2.2 to 2.2.3 in /sdks/python/container/py310

2023-02-15 Thread via GitHub
dependabot[bot] opened a new pull request, #25492: URL: https://github.com/apache/beam/pull/25492 Bumps [werkzeug](https://github.com/pallets/werkzeug) from 2.2.2 to 2.2.3. Release notes Sourced from https://github.com/pallets/werkzeug/releases";>werkzeug's releases. 2.2.3

[GitHub] [beam] damccorm merged pull request #25368: [Python] Added Tensorflow Model Handler

2023-02-15 Thread via GitHub
damccorm merged PR #25368: URL: https://github.com/apache/beam/pull/25368 -- 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...@beam.apache

[GitHub] [beam] damccorm commented on a diff in pull request #25368: [Python] Added Tensorflow Model Handler

2023-02-15 Thread via GitHub
damccorm commented on code in PR #25368: URL: https://github.com/apache/beam/pull/25368#discussion_r1107674009 ## sdks/python/apache_beam/ml/inference/tensorflow_inference.py: ## @@ -0,0 +1,253 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contri

[GitHub] [beam] ahmedabu98 commented on pull request #25472: One formatter for Timestamp fields in Storage writes

2023-02-15 Thread via GitHub
ahmedabu98 commented on PR #25472: URL: https://github.com/apache/beam/pull/25472#issuecomment-1431961534 @johnjcasey, @an2x read for review -- 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] ahmedabu98 commented on a diff in pull request #25472: One formatter for Timestamp fields in Storage writes

2023-02-15 Thread via GitHub
ahmedabu98 commented on code in PR #25472: URL: https://github.com/apache/beam/pull/25472#discussion_r1107665737 ## sdks/java/io/google-cloud-platform/src/main/java/org/apache/beam/sdk/io/gcp/bigquery/TableRowToStorageApiProto.java: ## @@ -80,11 +81,32 @@ public class TableRowTo

[GitHub] [beam] kennknowles commented on issue #25485: [Failing Test]: GroupByKeyTest$BasicTests$.testAfterProcessingTimeContinuationTriggerUsingState()

2023-02-15 Thread via GitHub
kennknowles commented on issue #25485: URL: https://github.com/apache/beam/issues/25485#issuecomment-1431955011 I spent a while to trace your logic, and it makes sense to me. I'm not sure the best way to proceed. I hate tests with delays or inherent races in them, but of course this one is

[GitHub] [beam] johannaojeling commented on a diff in pull request #25437: [Go SDK]: Allow SDF methods to have context param and error return value

2023-02-15 Thread via GitHub
johannaojeling commented on code in PR #25437: URL: https://github.com/apache/beam/pull/25437#discussion_r1107656944 ## sdks/go/pkg/beam/core/runtime/exec/sdf_invokers.go: ## @@ -39,61 +43,45 @@ import ( // cirInvoker is an invoker for CreateInitialRestriction. type cirInvok

[GitHub] [beam] pabloem merged pull request #25491: Fix provider to be found by AutoService

2023-02-15 Thread via GitHub
pabloem merged PR #25491: URL: https://github.com/apache/beam/pull/25491 -- 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...@beam.apache.

[GitHub] [beam] pabloem commented on pull request #25491: Fix provider to be found by AutoService

2023-02-15 Thread via GitHub
pabloem commented on PR #25491: URL: https://github.com/apache/beam/pull/25491#issuecomment-1431937441 only license pull failed in 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 sp

[GitHub] [beam] pabloem opened a new pull request, #25491: Fix provider to be found by AutoService

2023-02-15 Thread via GitHub
pabloem opened a new pull request, #25491: URL: https://github.com/apache/beam/pull/25491 r: @damondouglas Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily: - [ ] Mention the app

[GitHub] [beam] MakarkinSAkvelon closed pull request #25490: [Playground] Egress configuration for background, MiTMproxy solution

2023-02-15 Thread via GitHub
MakarkinSAkvelon closed pull request #25490: [Playground] Egress configuration for background, MiTMproxy solution URL: https://github.com/apache/beam/pull/25490 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abov

[GitHub] [beam] MakarkinSAkvelon opened a new pull request, #25490: [Playground] Egress configuration for background, MiTMproxy solution

2023-02-15 Thread via GitHub
MakarkinSAkvelon opened a new pull request, #25490: URL: https://github.com/apache/beam/pull/25490 1. Direct access from backend pods to the internet will be removed 2. Whitelist for backend pods 3. MiTMproxy configuration 4. Network policy for GKE Cluster

[GitHub] [beam] pabloem merged pull request #25479: SpannerIO: Handling pg spanner.commit_timestamp

2023-02-15 Thread via GitHub
pabloem merged PR #25479: URL: https://github.com/apache/beam/pull/25479 -- 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...@beam.apache.

[GitHub] [beam] pabloem commented on pull request #25479: SpannerIO: Handling pg spanner.commit_timestamp

2023-02-15 Thread via GitHub
pabloem commented on PR #25479: URL: https://github.com/apache/beam/pull/25479#issuecomment-1431849996 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. To unsubscribe,

[GitHub] [beam] riteshghorse commented on pull request #25488: [Python] fix pipeline parsing with single dash

2023-02-15 Thread via GitHub
riteshghorse commented on PR #25488: URL: https://github.com/apache/beam/pull/25488#issuecomment-1431776749 we currently consider `--unknown_option=some_value` & `-unknown_option=some_value` as same and that's why tests are failing -- This is an automated message from the Apache Git Serv

[GitHub] [beam] alexeyinkin opened a new issue, #25489: [Task]: Do not URL-encode links when sharing a catalog example in Playground

2023-02-15 Thread via GitHub
alexeyinkin opened a new issue, #25489: URL: https://github.com/apache/beam/issues/25489 ### What needs to happen? When clicking "Share my code" on an unmodified catalog example, the shareable link is in the following format: ``` https://play-dev.beam.apache.org/?path=SDK_J

[GitHub] [beam] Abacn commented on pull request #24992: [Avro] Use "extensions/avro" instead of avro from"core" in Java SDK modules

2023-02-15 Thread via GitHub
Abacn commented on PR #24992: URL: https://github.com/apache/beam/pull/24992#issuecomment-1431750213 > @Abacn Thanks for a ping. I think this PR is almost ready to be merged. In the mean time, the next step will be a deprecation of Avro classes in "sdks/java/core" but before, I think we nee

[GitHub] [beam] damccorm merged pull request #25487: [Playground][Tour of Beam] Use example id as CloudPath

2023-02-15 Thread via GitHub
damccorm merged PR #25487: URL: https://github.com/apache/beam/pull/25487 -- 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...@beam.apache

[GitHub] [beam] pabloem commented on pull request #25236: SpannerIO: parameterizing partitionQuery timeout

2023-02-15 Thread via GitHub
pabloem commented on PR #25236: URL: https://github.com/apache/beam/pull/25236#issuecomment-1431672905 Discussed offline. There are non-flex Spanner templates that the team wants to support. LGTM -- This is an automated message from the Apache Git Service. To respond to the message

[GitHub] [beam] pabloem merged pull request #25236: SpannerIO: parameterizing partitionQuery timeout

2023-02-15 Thread via GitHub
pabloem merged PR #25236: URL: https://github.com/apache/beam/pull/25236 -- 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...@beam.apache.

[GitHub] [beam] pabloem closed issue #25025: [Feature Request]: Increase Spanner PartitionQuery timeout to 5 minutes

2023-02-15 Thread via GitHub
pabloem closed issue #25025: [Feature Request]: Increase Spanner PartitionQuery timeout to 5 minutes URL: https://github.com/apache/beam/issues/25025 -- 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

[GitHub] [beam] pabloem merged pull request #25474: Not including support for end time in Bigtable Change Stream connector

2023-02-15 Thread via GitHub
pabloem merged PR #25474: URL: https://github.com/apache/beam/pull/25474 -- 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...@beam.apache.

[GitHub] [beam] pabloem commented on pull request #25474: Not including support for end time in Bigtable Change Stream connector

2023-02-15 Thread via GitHub
pabloem commented on PR #25474: URL: https://github.com/apache/beam/pull/25474#issuecomment-1431671470 mmerging for now -- 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 u

[GitHub] [beam] pabloem commented on pull request #25474: Not including support for end time in Bigtable Change Stream connector

2023-02-15 Thread via GitHub
pabloem commented on PR #25474: URL: https://github.com/apache/beam/pull/25474#issuecomment-1431671279 it's a little hard to find, but precommit is failing on `pullLicenses` task, which was broken on master when the branch was started: ![image](https://user-images.githubusercontent.c

[GitHub] [beam] pabloem commented on a diff in pull request #25236: SpannerIO: parameterizing partitionQuery timeout

2023-02-15 Thread via GitHub
pabloem commented on code in PR #25236: URL: https://github.com/apache/beam/pull/25236#discussion_r1107381512 ## sdks/java/io/google-cloud-platform/src/main/java/org/apache/beam/sdk/io/gcp/spanner/SpannerConfig.java: ## @@ -265,4 +273,24 @@ public SpannerConfig withRpcPriority(

[GitHub] [beam] pabloem commented on pull request #25236: SpannerIO: parameterizing partitionQuery timeout

2023-02-15 Thread via GitHub
pabloem commented on PR #25236: URL: https://github.com/apache/beam/pull/25236#issuecomment-1431665012 we no longer use ValuProvider parameters for transforms. These werre used for 'old' template formats, but they are no longer necessary for 'Flex' templates which is the main kind of templa

[GitHub] [beam] aromanenko-dev commented on pull request #24992: [Avro] Use "extensions/avro" instead of avro from"core" in Java SDK modules

2023-02-15 Thread via GitHub
aromanenko-dev commented on PR #24992: URL: https://github.com/apache/beam/pull/24992#issuecomment-1431656386 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 comm

[GitHub] [beam] tonytanger commented on pull request #25474: Not including support for end time in Bigtable Change Stream connector

2023-02-15 Thread via GitHub
tonytanger commented on PR #25474: URL: https://github.com/apache/beam/pull/25474#issuecomment-1431646888 Java PreCommit shows no issues. Not sure why it says the build failed. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub a

[GitHub] [beam] riteshghorse commented on pull request #25488: [Python] fix pipeline parsing with single dash

2023-02-15 Thread via GitHub
riteshghorse commented on PR #25488: URL: https://github.com/apache/beam/pull/25488#issuecomment-1431645914 R: @tvalentyn -- 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. T

[GitHub] [beam] riteshghorse opened a new pull request, #25488: [Python] fix pipeline parsing with single dash

2023-02-15 Thread via GitHub
riteshghorse opened a new pull request, #25488: URL: https://github.com/apache/beam/pull/25488 This PR prevents misinterpreting of `-f` as `--f` which is a valid shorthand of `--flexrs_goal` and similar. Thank you for your contribution! Follow this che

[GitHub] [beam] aromanenko-dev commented on pull request #24992: [Avro] Use "extensions/avro" instead of avro from"core" in Java SDK modules

2023-02-15 Thread via GitHub
aromanenko-dev commented on PR #24992: URL: https://github.com/apache/beam/pull/24992#issuecomment-1431640265 @Abacn Thanks for a ping. I think this PR is almost ready to be merged. In the mean time, the next step will be a deprecation of Avro classes in "sdks/java/core" but before, I think

  1   2   >