You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@beam.apache.org by GitBox <gi...@apache.org> on 2021/03/04 05:20:25 UTC
[GitHub] [beam] y1chi opened a new pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
y1chi opened a new pull request #14145:
URL: https://github.com/apache/beam/pull/14145
…Cont
**Please** add a meaningful description for your change here
------------------------
Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:
- [ ] [**Choose reviewer(s)**](https://beam.apache.org/contribute/#make-your-change) and mention them in a comment (`R: @username`).
- [ ] Format the pull request title like `[BEAM-XXX] Fixes bug in ApproximateQuantiles`, where you replace `BEAM-XXX` with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
- [ ] Update `CHANGES.md` with noteworthy changes.
- [ ] If this contribution is large, please file an Apache [Individual Contributor License Agreement](https://www.apache.org/licenses/icla.pdf).
See the [Contributor Guide](https://beam.apache.org/contribute) for more tips on [how to make review process smoother](https://beam.apache.org/contribute/#make-reviewers-job-easier).
Post-Commit Tests Status (on master branch)
------------------------------------------------------------------------------------------------
Lang | SDK | Dataflow | Flink | Samza | Spark | Twister2
--- | --- | --- | --- | --- | --- | ---
Go | [](https://ci-beam.apache.org/job/beam_PostCommit_Go/lastCompletedBuild/) | --- | [](https://ci-beam.apache.org/job/beam_PostCommit_Go_VR_Flink/lastCompletedBuild/) | --- | [](https://ci-beam.apache.org/job/beam_PostCommit_Go_VR_Spark/lastCompletedBuild/) | ---
Java | [](https://ci-beam.apache.org/job/beam_PostCommit_Java/lastCompletedBuild/) | [](https://ci-beam.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Dataflow/lastCompletedBuild/)<br>[](https://ci-beam.apache.org/job/beam_PostCommit_Java_VR_Dataflow_V2/lastCompletedBuild/)<br>[](https://ci-beam.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Dataflow_Java11/lastCompletedBuild/) | [](https://ci-beam
.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Flink/lastCompletedBuild/)<br>[](https://ci-beam.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Flink_Java11/lastCompletedBuild/)<br>[](https://ci-beam.apache.org/job/beam_PostCommit_Java_PVR_Flink_Batch/lastCompletedBuild/)<br>[](https://ci-beam.apache.org/job/beam_PostCommit_Java_PVR_Flink_Streaming/lastCompletedBuild/) | [](https://ci-beam.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Samza/lastCompletedBuild/) | [](https://ci-beam.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Spark/lastCompletedBuild/)<br>[](https://ci-beam.apache.org/job/beam_PostCommit_Java_PVR_Spark_Batch/lastCompletedBuild/)<br>[](https://ci-beam.apache.org/job/beam_PostCommit_Java_ValidatesRunner_SparkStructuredStreaming/lastCompletedBuild/) | [](https://ci-beam.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Twister2/lastCompletedBuild/)
Python | [](https://ci-beam.apache.org/job/beam_PostCommit_Python36/lastCompletedBuild/)<br>[](https://ci-beam.apache.org/job/beam_PostCommit_Python37/lastCompletedBuild/)<br>[](https://ci-beam.apache.org/job/beam_PostCommit_Python38/lastCompletedBuild/) | [](https://ci-beam.apache.org/job/beam_PostCommit_Py_VR_Dataflow/lastCompletedBuild/)<br>[](https://ci-beam.apache.org/job/beam_PostCommit_Py_VR_Dataflow_V2/lastCompletedBuild/)<br>[](https://ci-beam.apache.org/job/beam_PostCommit_Py_ValCont/lastCompletedBuild/) | [](https://ci-beam.apache.org/job/beam_PreCommit_Python_PVR_Flink_Cron/lastCompletedBuild/)<br>[](https://ci-beam.apache.org/job/beam_PostCommit_Python_VR_Flink/lastCompletedBuild/) | --- | [](https://ci-beam.apache.org/job/beam_PostCommit_Python_VR_Spark/lastCompletedBuild/) | ---
XLang | [](https://ci-beam.apache.org/job/beam_PostCommit_XVR_Direct/lastCompletedBuild/) | [](https://ci-beam.apache.org/job/beam_PostCommit_XVR_Dataflow/lastCompletedBuild/) | [](https://ci-beam.apache.org/job/beam_PostCommit_XVR_Flink/lastCompletedBuild/) | --- | [](https://ci-beam.apache.org/job/beam_PostCommit_XVR_Spark/lastCompletedBuild/) | ---
Pre-Commit Tests Status (on master branch)
------------------------------------------------------------------------------------------------
--- |Java | Python | Go | Website | Whitespace | Typescript
--- | --- | --- | --- | --- | --- | ---
Non-portable | [](https://ci-beam.apache.org/job/beam_PreCommit_Java_Cron/lastCompletedBuild/) | [](https://ci-beam.apache.org/job/beam_PreCommit_Python_Cron/lastCompletedBuild/)<br>[](https://ci-beam.apache.org/job/beam_PreCommit_PythonLint_Cron/lastCompletedBuild/)<br>[](https://ci-beam.apache.org/job/beam_PreCommit_PythonDocker_Cron/lastCompletedBuild/) <br>[](https://ci-beam.apache.org/job/beam_PreCommit_PythonDocs_Cron/lastCompletedBuild/) | [](https://ci-beam.apache.org/job/beam_PreCommit_Go_Cron/lastCompletedBuild/) | [](https://ci-beam.apache.org/job/beam_PreCommit_Website_Cron/lastCompletedBuild/) | [](https://ci-beam.apache.org/job/beam_PreCommit_Whitespace_Cron/lastCompletedBuild/) | [](https://ci-beam.apache.org/job/beam_PreCommit_Typescript_Cron/lastCompletedBuild/)
Portable | --- | [](https://ci-beam.apache.org/job/beam_PreCommit_Portable_Python_Cron/lastCompletedBuild/) | --- | --- | --- | ---
See [.test-infra/jenkins/README](https://github.com/apache/beam/blob/master/.test-infra/jenkins/README.md) for trigger phrase, status and link of all Jenkins jobs.
GitHub Actions Tests Status (on master branch)
------------------------------------------------------------------------------------------------
[](https://github.com/apache/beam/actions?query=workflow%3A%22Build+python+source+distribution+and+wheels%22+branch%3Amaster+event%3Aschedule)
[](https://github.com/apache/beam/actions?query=workflow%3A%22Python+Tests%22+branch%3Amaster+event%3Aschedule)
[](https://github.com/apache/beam/actions?query=workflow%3A%22Java+Tests%22+branch%3Amaster+event%3Aschedule)
See [CI.md](https://github.com/apache/beam/blob/master/CI.md) for more information about GitHub Actions CI.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] codecov[bot] commented on pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
codecov[bot] commented on pull request #14145:
URL: https://github.com/apache/beam/pull/14145#issuecomment-790308850
# [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=h1) Report
> Merging [#14145](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=desc) (15e2d0b) into [master](https://codecov.io/gh/apache/beam/commit/2d9c66647ecbc9aca6a8618168d44e4c815cea19?el=desc) (2d9c666) will **decrease** coverage by `0.00%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #14145 +/- ##
==========================================
- Coverage 83.04% 83.04% -0.01%
==========================================
Files 469 469
Lines 58472 58473 +1
==========================================
Hits 48556 48556
- Misses 9916 9917 +1
```
| [Impacted Files](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [...38/build/srcs/sdks/python/apache\_beam/io/iobase.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9pby9pb2Jhc2UucHk=) | `84.55% <0.00%> (-0.27%)` | :arrow_down: |
| [...sdks/python/apache\_beam/transforms/environments.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS90cmFuc2Zvcm1zL2Vudmlyb25tZW50cy5weQ==) | `85.24% <0.00%> (+0.04%)` | :arrow_up: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=footer). Last update [1f35e2f...15e2d0b](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] tvalentyn commented on pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
tvalentyn commented on pull request #14145:
URL: https://github.com/apache/beam/pull/14145#issuecomment-791718778
Run Python Dataflow ValidatesContainer
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] tvalentyn commented on pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
tvalentyn commented on pull request #14145:
URL: https://github.com/apache/beam/pull/14145#issuecomment-791088534
LGTM as long as the suite (https://ci-beam.apache.org/job/beam_PostCommit_Py_ValCont_PR/162/) passes, thanks @y1chi !
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] y1chi commented on pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
y1chi commented on pull request #14145:
URL: https://github.com/apache/beam/pull/14145#issuecomment-791701376
@tvalentyn @angoenka could you try to rerun the seed job and validate container job again?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] codecov[bot] edited a comment on pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
codecov[bot] edited a comment on pull request #14145:
URL: https://github.com/apache/beam/pull/14145#issuecomment-790308850
# [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=h1) Report
> Merging [#14145](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=desc) (b473163) into [master](https://codecov.io/gh/apache/beam/commit/2d9c66647ecbc9aca6a8618168d44e4c815cea19?el=desc) (2d9c666) will **increase** coverage by `0.12%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #14145 +/- ##
==========================================
+ Coverage 83.04% 83.16% +0.12%
==========================================
Files 469 469
Lines 58472 58954 +482
==========================================
+ Hits 48556 49029 +473
- Misses 9916 9925 +9
```
| [Impacted Files](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [.../runners/interactive/testing/test\_cache\_manager.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL2ludGVyYWN0aXZlL3Rlc3RpbmcvdGVzdF9jYWNoZV9tYW5hZ2VyLnB5) | `91.30% <0.00%> (-2.90%)` | :arrow_down: |
| [.../python/apache\_beam/transforms/periodicsequence.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS90cmFuc2Zvcm1zL3BlcmlvZGljc2VxdWVuY2UucHk=) | `96.49% <0.00%> (-1.76%)` | :arrow_down: |
| [.../internal/clients/dataflow/dataflow\_v1b3\_client.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL2RhdGFmbG93L2ludGVybmFsL2NsaWVudHMvZGF0YWZsb3cvZGF0YWZsb3dfdjFiM19jbGllbnQucHk=) | `71.34% <0.00%> (-1.47%)` | :arrow_down: |
| [...cs/sdks/python/apache\_beam/dataframe/frame\_base.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9kYXRhZnJhbWUvZnJhbWVfYmFzZS5weQ==) | `89.70% <0.00%> (-0.80%)` | :arrow_down: |
| [...on/apache\_beam/runners/portability/spark\_runner.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL3BvcnRhYmlsaXR5L3NwYXJrX3J1bm5lci5weQ==) | `55.81% <0.00%> (-0.71%)` | :arrow_down: |
| [.../srcs/sdks/python/apache\_beam/dataframe/convert.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9kYXRhZnJhbWUvY29udmVydC5weQ==) | `90.47% <0.00%> (-0.44%)` | :arrow_down: |
| [...on/apache\_beam/runners/portability/flink\_runner.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL3BvcnRhYmlsaXR5L2ZsaW5rX3J1bm5lci5weQ==) | `61.11% <0.00%> (-0.30%)` | :arrow_down: |
| [...srcs/sdks/python/apache\_beam/dataframe/doctests.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9kYXRhZnJhbWUvZG9jdGVzdHMucHk=) | `97.55% <0.00%> (-0.15%)` | :arrow_down: |
| [...ks/python/apache\_beam/runners/worker/operations.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL3dvcmtlci9vcGVyYXRpb25zLnB5) | `70.27% <0.00%> (-0.11%)` | :arrow_down: |
| [...on/apache\_beam/runners/dataflow/dataflow\_runner.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL2RhdGFmbG93L2RhdGFmbG93X3J1bm5lci5weQ==) | `81.83% <0.00%> (-0.07%)` | :arrow_down: |
| ... and [23 more](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=footer). Last update [1f35e2f...b473163](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] y1chi commented on pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
y1chi commented on pull request #14145:
URL: https://github.com/apache/beam/pull/14145#issuecomment-790303949
Run seed job
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] codecov[bot] edited a comment on pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
codecov[bot] edited a comment on pull request #14145:
URL: https://github.com/apache/beam/pull/14145#issuecomment-790308850
# [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=h1) Report
> Merging [#14145](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=desc) (b473163) into [master](https://codecov.io/gh/apache/beam/commit/2d9c66647ecbc9aca6a8618168d44e4c815cea19?el=desc) (2d9c666) will **increase** coverage by `0.12%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #14145 +/- ##
==========================================
+ Coverage 83.04% 83.16% +0.12%
==========================================
Files 469 469
Lines 58472 58954 +482
==========================================
+ Hits 48556 49029 +473
- Misses 9916 9925 +9
```
| [Impacted Files](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [.../runners/interactive/testing/test\_cache\_manager.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL2ludGVyYWN0aXZlL3Rlc3RpbmcvdGVzdF9jYWNoZV9tYW5hZ2VyLnB5) | `91.30% <0.00%> (-2.90%)` | :arrow_down: |
| [.../python/apache\_beam/transforms/periodicsequence.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS90cmFuc2Zvcm1zL3BlcmlvZGljc2VxdWVuY2UucHk=) | `96.49% <0.00%> (-1.76%)` | :arrow_down: |
| [.../internal/clients/dataflow/dataflow\_v1b3\_client.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL2RhdGFmbG93L2ludGVybmFsL2NsaWVudHMvZGF0YWZsb3cvZGF0YWZsb3dfdjFiM19jbGllbnQucHk=) | `71.34% <0.00%> (-1.47%)` | :arrow_down: |
| [...cs/sdks/python/apache\_beam/dataframe/frame\_base.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9kYXRhZnJhbWUvZnJhbWVfYmFzZS5weQ==) | `89.70% <0.00%> (-0.80%)` | :arrow_down: |
| [...on/apache\_beam/runners/portability/spark\_runner.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL3BvcnRhYmlsaXR5L3NwYXJrX3J1bm5lci5weQ==) | `55.81% <0.00%> (-0.71%)` | :arrow_down: |
| [.../srcs/sdks/python/apache\_beam/dataframe/convert.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9kYXRhZnJhbWUvY29udmVydC5weQ==) | `90.47% <0.00%> (-0.44%)` | :arrow_down: |
| [...on/apache\_beam/runners/portability/flink\_runner.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL3BvcnRhYmlsaXR5L2ZsaW5rX3J1bm5lci5weQ==) | `61.11% <0.00%> (-0.30%)` | :arrow_down: |
| [...srcs/sdks/python/apache\_beam/dataframe/doctests.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9kYXRhZnJhbWUvZG9jdGVzdHMucHk=) | `97.55% <0.00%> (-0.15%)` | :arrow_down: |
| [...ks/python/apache\_beam/runners/worker/operations.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL3dvcmtlci9vcGVyYXRpb25zLnB5) | `70.27% <0.00%> (-0.11%)` | :arrow_down: |
| [...on/apache\_beam/runners/dataflow/dataflow\_runner.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL2RhdGFmbG93L2RhdGFmbG93X3J1bm5lci5weQ==) | `81.83% <0.00%> (-0.07%)` | :arrow_down: |
| ... and [23 more](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=footer). Last update [1f35e2f...b473163](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] codecov[bot] edited a comment on pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
codecov[bot] edited a comment on pull request #14145:
URL: https://github.com/apache/beam/pull/14145#issuecomment-790308850
# [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=h1) Report
> Merging [#14145](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=desc) (04ea4e3) into [master](https://codecov.io/gh/apache/beam/commit/2d9c66647ecbc9aca6a8618168d44e4c815cea19?el=desc) (2d9c666) will **increase** coverage by `0.01%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #14145 +/- ##
==========================================
+ Coverage 83.04% 83.05% +0.01%
==========================================
Files 469 469
Lines 58472 58493 +21
==========================================
+ Hits 48556 48580 +24
+ Misses 9916 9913 -3
```
| [Impacted Files](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [.../runners/interactive/testing/test\_cache\_manager.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL2ludGVyYWN0aXZlL3Rlc3RpbmcvdGVzdF9jYWNoZV9tYW5hZ2VyLnB5) | `91.30% <0.00%> (-2.90%)` | :arrow_down: |
| [...sdks/python/apache\_beam/transforms/environments.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS90cmFuc2Zvcm1zL2Vudmlyb25tZW50cy5weQ==) | `85.24% <0.00%> (+0.04%)` | :arrow_up: |
| [...srcs/sdks/python/apache\_beam/dataframe/doctests.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9kYXRhZnJhbWUvZG9jdGVzdHMucHk=) | `97.74% <0.00%> (+0.05%)` | :arrow_up: |
| [...d/srcs/sdks/python/apache\_beam/dataframe/frames.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9kYXRhZnJhbWUvZnJhbWVzLnB5) | `91.00% <0.00%> (+0.20%)` | :arrow_up: |
| [...eam/runners/interactive/interactive\_environment.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL2ludGVyYWN0aXZlL2ludGVyYWN0aXZlX2Vudmlyb25tZW50LnB5) | `90.28% <0.00%> (+0.35%)` | :arrow_up: |
| [...hon/apache\_beam/runners/direct/test\_stream\_impl.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL2RpcmVjdC90ZXN0X3N0cmVhbV9pbXBsLnB5) | `94.11% <0.00%> (+2.20%)` | :arrow_up: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=footer). Last update [1f35e2f...04ea4e3](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] codecov[bot] edited a comment on pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
codecov[bot] edited a comment on pull request #14145:
URL: https://github.com/apache/beam/pull/14145#issuecomment-790308850
# [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=h1) Report
> Merging [#14145](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=desc) (9cbc36c) into [master](https://codecov.io/gh/apache/beam/commit/2d9c66647ecbc9aca6a8618168d44e4c815cea19?el=desc) (2d9c666) will **increase** coverage by `0.00%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #14145 +/- ##
=======================================
Coverage 83.04% 83.04%
=======================================
Files 469 469
Lines 58472 58493 +21
=======================================
+ Hits 48556 48577 +21
Misses 9916 9916
```
| [Impacted Files](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [.../runners/interactive/testing/test\_cache\_manager.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL2ludGVyYWN0aXZlL3Rlc3RpbmcvdGVzdF9jYWNoZV9tYW5hZ2VyLnB5) | `91.30% <0.00%> (-2.90%)` | :arrow_down: |
| [...sdks/python/apache\_beam/transforms/environments.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS90cmFuc2Zvcm1zL2Vudmlyb25tZW50cy5weQ==) | `85.24% <0.00%> (+0.04%)` | :arrow_up: |
| [...srcs/sdks/python/apache\_beam/dataframe/doctests.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9kYXRhZnJhbWUvZG9jdGVzdHMucHk=) | `97.74% <0.00%> (+0.05%)` | :arrow_up: |
| [...d/srcs/sdks/python/apache\_beam/dataframe/frames.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9kYXRhZnJhbWUvZnJhbWVzLnB5) | `91.00% <0.00%> (+0.20%)` | :arrow_up: |
| [...eam/runners/interactive/interactive\_environment.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL2ludGVyYWN0aXZlL2ludGVyYWN0aXZlX2Vudmlyb25tZW50LnB5) | `90.28% <0.00%> (+0.35%)` | :arrow_up: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=footer). Last update [1f35e2f...3ed74e8](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] codecov[bot] edited a comment on pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
codecov[bot] edited a comment on pull request #14145:
URL: https://github.com/apache/beam/pull/14145#issuecomment-790308850
# [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=h1) Report
> Merging [#14145](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=desc) (04ea4e3) into [master](https://codecov.io/gh/apache/beam/commit/2d9c66647ecbc9aca6a8618168d44e4c815cea19?el=desc) (2d9c666) will **increase** coverage by `0.01%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #14145 +/- ##
==========================================
+ Coverage 83.04% 83.05% +0.01%
==========================================
Files 469 469
Lines 58472 58493 +21
==========================================
+ Hits 48556 48580 +24
+ Misses 9916 9913 -3
```
| [Impacted Files](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [.../runners/interactive/testing/test\_cache\_manager.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL2ludGVyYWN0aXZlL3Rlc3RpbmcvdGVzdF9jYWNoZV9tYW5hZ2VyLnB5) | `91.30% <0.00%> (-2.90%)` | :arrow_down: |
| [...sdks/python/apache\_beam/transforms/environments.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS90cmFuc2Zvcm1zL2Vudmlyb25tZW50cy5weQ==) | `85.24% <0.00%> (+0.04%)` | :arrow_up: |
| [...srcs/sdks/python/apache\_beam/dataframe/doctests.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9kYXRhZnJhbWUvZG9jdGVzdHMucHk=) | `97.74% <0.00%> (+0.05%)` | :arrow_up: |
| [...d/srcs/sdks/python/apache\_beam/dataframe/frames.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9kYXRhZnJhbWUvZnJhbWVzLnB5) | `91.00% <0.00%> (+0.20%)` | :arrow_up: |
| [...eam/runners/interactive/interactive\_environment.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL2ludGVyYWN0aXZlL2ludGVyYWN0aXZlX2Vudmlyb25tZW50LnB5) | `90.28% <0.00%> (+0.35%)` | :arrow_up: |
| [...hon/apache\_beam/runners/direct/test\_stream\_impl.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL2RpcmVjdC90ZXN0X3N0cmVhbV9pbXBsLnB5) | `94.11% <0.00%> (+2.20%)` | :arrow_up: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=footer). Last update [1f35e2f...04ea4e3](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] codecov[bot] edited a comment on pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
codecov[bot] edited a comment on pull request #14145:
URL: https://github.com/apache/beam/pull/14145#issuecomment-790308850
# [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=h1) Report
> Merging [#14145](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=desc) (a59293d) into [master](https://codecov.io/gh/apache/beam/commit/2d9c66647ecbc9aca6a8618168d44e4c815cea19?el=desc) (2d9c666) will **increase** coverage by `0.00%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #14145 +/- ##
=======================================
Coverage 83.04% 83.05%
=======================================
Files 469 469
Lines 58472 58493 +21
=======================================
+ Hits 48556 48579 +23
+ Misses 9916 9914 -2
```
| [Impacted Files](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [...teractive/testing/integration/notebook\_executor.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL2ludGVyYWN0aXZlL3Rlc3RpbmcvaW50ZWdyYXRpb24vbm90ZWJvb2tfZXhlY3V0b3IucHk=) | | |
| [...ld/srcs/sdks/python/apache\_beam/coders/\_\_init\_\_.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9jb2RlcnMvX19pbml0X18ucHk=) | | |
| [...s/python/apache\_beam/examples/snippets/snippets.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9leGFtcGxlcy9zbmlwcGV0cy9zbmlwcGV0cy5weQ==) | | |
| [...on/apache\_beam/examples/complete/juliaset/setup.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9leGFtcGxlcy9jb21wbGV0ZS9qdWxpYXNldC9zZXR1cC5weQ==) | | |
| [...cs/sdks/python/apache\_beam/transforms/combiners.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS90cmFuc2Zvcm1zL2NvbWJpbmVycy5weQ==) | | |
| [...python/apache\_beam/examples/complete/distribopt.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9leGFtcGxlcy9jb21wbGV0ZS9kaXN0cmlib3B0LnB5) | | |
| [.../sdks/python/apache\_beam/internal/metrics/cells.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9pbnRlcm5hbC9tZXRyaWNzL2NlbGxzLnB5) | | |
| [.../python/apache\_beam/testing/benchmarks/\_\_init\_\_.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS90ZXN0aW5nL2JlbmNobWFya3MvX19pbml0X18ucHk=) | | |
| [...d/srcs/sdks/python/apache\_beam/coders/row\_coder.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9jb2RlcnMvcm93X2NvZGVyLnB5) | | |
| [...y38/build/srcs/sdks/python/apache\_beam/pipeline.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9waXBlbGluZS5weQ==) | | |
| ... and [928 more](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=footer). Last update [1f35e2f...a59293d](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] codecov[bot] edited a comment on pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
codecov[bot] edited a comment on pull request #14145:
URL: https://github.com/apache/beam/pull/14145#issuecomment-790308850
# [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=h1) Report
> Merging [#14145](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=desc) (04ea4e3) into [master](https://codecov.io/gh/apache/beam/commit/2d9c66647ecbc9aca6a8618168d44e4c815cea19?el=desc) (2d9c666) will **increase** coverage by `0.01%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #14145 +/- ##
==========================================
+ Coverage 83.04% 83.05% +0.01%
==========================================
Files 469 469
Lines 58472 58493 +21
==========================================
+ Hits 48556 48580 +24
+ Misses 9916 9913 -3
```
| [Impacted Files](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [.../runners/interactive/testing/test\_cache\_manager.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL2ludGVyYWN0aXZlL3Rlc3RpbmcvdGVzdF9jYWNoZV9tYW5hZ2VyLnB5) | `91.30% <0.00%> (-2.90%)` | :arrow_down: |
| [...sdks/python/apache\_beam/transforms/environments.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS90cmFuc2Zvcm1zL2Vudmlyb25tZW50cy5weQ==) | `85.24% <0.00%> (+0.04%)` | :arrow_up: |
| [...srcs/sdks/python/apache\_beam/dataframe/doctests.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9kYXRhZnJhbWUvZG9jdGVzdHMucHk=) | `97.74% <0.00%> (+0.05%)` | :arrow_up: |
| [...d/srcs/sdks/python/apache\_beam/dataframe/frames.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9kYXRhZnJhbWUvZnJhbWVzLnB5) | `91.00% <0.00%> (+0.20%)` | :arrow_up: |
| [...eam/runners/interactive/interactive\_environment.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL2ludGVyYWN0aXZlL2ludGVyYWN0aXZlX2Vudmlyb25tZW50LnB5) | `90.28% <0.00%> (+0.35%)` | :arrow_up: |
| [...hon/apache\_beam/runners/direct/test\_stream\_impl.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL2RpcmVjdC90ZXN0X3N0cmVhbV9pbXBsLnB5) | `94.11% <0.00%> (+2.20%)` | :arrow_up: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=footer). Last update [1f35e2f...04ea4e3](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] codecov[bot] edited a comment on pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
codecov[bot] edited a comment on pull request #14145:
URL: https://github.com/apache/beam/pull/14145#issuecomment-790308850
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] tvalentyn commented on pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
tvalentyn commented on pull request #14145:
URL: https://github.com/apache/beam/pull/14145#issuecomment-793188754
rerunning seed job just in case
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] codecov[bot] edited a comment on pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
codecov[bot] edited a comment on pull request #14145:
URL: https://github.com/apache/beam/pull/14145#issuecomment-790308850
# [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=h1) Report
> Merging [#14145](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=desc) (9cbc36c) into [master](https://codecov.io/gh/apache/beam/commit/2d9c66647ecbc9aca6a8618168d44e4c815cea19?el=desc) (2d9c666) will **increase** coverage by `0.00%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #14145 +/- ##
=======================================
Coverage 83.04% 83.04%
=======================================
Files 469 469
Lines 58472 58493 +21
=======================================
+ Hits 48556 48577 +21
Misses 9916 9916
```
| [Impacted Files](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [.../runners/interactive/testing/test\_cache\_manager.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL2ludGVyYWN0aXZlL3Rlc3RpbmcvdGVzdF9jYWNoZV9tYW5hZ2VyLnB5) | `91.30% <0.00%> (-2.90%)` | :arrow_down: |
| [...sdks/python/apache\_beam/transforms/environments.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS90cmFuc2Zvcm1zL2Vudmlyb25tZW50cy5weQ==) | `85.24% <0.00%> (+0.04%)` | :arrow_up: |
| [...srcs/sdks/python/apache\_beam/dataframe/doctests.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9kYXRhZnJhbWUvZG9jdGVzdHMucHk=) | `97.74% <0.00%> (+0.05%)` | :arrow_up: |
| [...d/srcs/sdks/python/apache\_beam/dataframe/frames.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9kYXRhZnJhbWUvZnJhbWVzLnB5) | `91.00% <0.00%> (+0.20%)` | :arrow_up: |
| [...eam/runners/interactive/interactive\_environment.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL2ludGVyYWN0aXZlL2ludGVyYWN0aXZlX2Vudmlyb25tZW50LnB5) | `90.28% <0.00%> (+0.35%)` | :arrow_up: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=footer). Last update [1f35e2f...04ea4e3](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] tvalentyn commented on pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
tvalentyn commented on pull request #14145:
URL: https://github.com/apache/beam/pull/14145#issuecomment-793199243
Run Python Dataflow ValidatesContainer
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] codecov[bot] edited a comment on pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
codecov[bot] edited a comment on pull request #14145:
URL: https://github.com/apache/beam/pull/14145#issuecomment-790308850
# [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=h1) Report
> Merging [#14145](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=desc) (b473163) into [master](https://codecov.io/gh/apache/beam/commit/2d9c66647ecbc9aca6a8618168d44e4c815cea19?el=desc) (2d9c666) will **increase** coverage by `0.12%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #14145 +/- ##
==========================================
+ Coverage 83.04% 83.16% +0.12%
==========================================
Files 469 469
Lines 58472 58954 +482
==========================================
+ Hits 48556 49029 +473
- Misses 9916 9925 +9
```
| [Impacted Files](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [.../runners/interactive/testing/test\_cache\_manager.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL2ludGVyYWN0aXZlL3Rlc3RpbmcvdGVzdF9jYWNoZV9tYW5hZ2VyLnB5) | `91.30% <0.00%> (-2.90%)` | :arrow_down: |
| [.../python/apache\_beam/transforms/periodicsequence.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS90cmFuc2Zvcm1zL3BlcmlvZGljc2VxdWVuY2UucHk=) | `96.49% <0.00%> (-1.76%)` | :arrow_down: |
| [.../internal/clients/dataflow/dataflow\_v1b3\_client.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL2RhdGFmbG93L2ludGVybmFsL2NsaWVudHMvZGF0YWZsb3cvZGF0YWZsb3dfdjFiM19jbGllbnQucHk=) | `71.34% <0.00%> (-1.47%)` | :arrow_down: |
| [...cs/sdks/python/apache\_beam/dataframe/frame\_base.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9kYXRhZnJhbWUvZnJhbWVfYmFzZS5weQ==) | `89.70% <0.00%> (-0.80%)` | :arrow_down: |
| [...on/apache\_beam/runners/portability/spark\_runner.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL3BvcnRhYmlsaXR5L3NwYXJrX3J1bm5lci5weQ==) | `55.81% <0.00%> (-0.71%)` | :arrow_down: |
| [.../srcs/sdks/python/apache\_beam/dataframe/convert.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9kYXRhZnJhbWUvY29udmVydC5weQ==) | `90.47% <0.00%> (-0.44%)` | :arrow_down: |
| [...on/apache\_beam/runners/portability/flink\_runner.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL3BvcnRhYmlsaXR5L2ZsaW5rX3J1bm5lci5weQ==) | `61.11% <0.00%> (-0.30%)` | :arrow_down: |
| [...srcs/sdks/python/apache\_beam/dataframe/doctests.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9kYXRhZnJhbWUvZG9jdGVzdHMucHk=) | `97.55% <0.00%> (-0.15%)` | :arrow_down: |
| [...ks/python/apache\_beam/runners/worker/operations.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL3dvcmtlci9vcGVyYXRpb25zLnB5) | `70.27% <0.00%> (-0.11%)` | :arrow_down: |
| [...on/apache\_beam/runners/dataflow/dataflow\_runner.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL2RhdGFmbG93L2RhdGFmbG93X3J1bm5lci5weQ==) | `81.83% <0.00%> (-0.07%)` | :arrow_down: |
| ... and [23 more](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=footer). Last update [1f35e2f...b473163](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] y1chi commented on pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
y1chi commented on pull request #14145:
URL: https://github.com/apache/beam/pull/14145#issuecomment-791167708
Run Python Dataflow ValidatesContainer
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] tvalentyn commented on a change in pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
tvalentyn commented on a change in pull request #14145:
URL: https://github.com/apache/beam/pull/14145#discussion_r592057739
##########
File path: sdks/python/container/run_validatescontainer.sh
##########
@@ -78,23 +82,26 @@ command -v gcloud
docker -v
gcloud -v
-# Build the container
+# Verify docker image has been built.
+docker images | grep "apache/$IMAGE_NAME" | grep "$SDK_VERSION"
Review comment:
Did you mean to do any verification logic here or this is debug output?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] codecov[bot] edited a comment on pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
codecov[bot] edited a comment on pull request #14145:
URL: https://github.com/apache/beam/pull/14145#issuecomment-790308850
# [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=h1) Report
> Merging [#14145](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=desc) (a59293d) into [master](https://codecov.io/gh/apache/beam/commit/2d9c66647ecbc9aca6a8618168d44e4c815cea19?el=desc) (2d9c666) will **increase** coverage by `0.00%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #14145 +/- ##
=======================================
Coverage 83.04% 83.05%
=======================================
Files 469 469
Lines 58472 58493 +21
=======================================
+ Hits 48556 48579 +23
+ Misses 9916 9914 -2
```
| [Impacted Files](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [...s/sdks/python/apache\_beam/io/gcp/tests/\_\_init\_\_.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9pby9nY3AvdGVzdHMvX19pbml0X18ucHk=) | | |
| [...apache\_beam/examples/cookbook/custom\_ptransform.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9leGFtcGxlcy9jb29rYm9vay9jdXN0b21fcHRyYW5zZm9ybS5weQ==) | | |
| [...che\_beam/examples/streaming\_wordcount\_debugging.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9leGFtcGxlcy9zdHJlYW1pbmdfd29yZGNvdW50X2RlYnVnZ2luZy5weQ==) | | |
| [...ld/srcs/sdks/python/apache\_beam/io/gcp/bigquery.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9pby9nY3AvYmlncXVlcnkucHk=) | | |
| [...e\_beam/runners/portability/abstract\_job\_service.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL3BvcnRhYmlsaXR5L2Fic3RyYWN0X2pvYl9zZXJ2aWNlLnB5) | | |
| [...s/python/apache\_beam/typehints/sharded\_key\_type.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS90eXBlaGludHMvc2hhcmRlZF9rZXlfdHlwZS5weQ==) | | |
| [...srcs/sdks/python/apache\_beam/io/filebasedsource.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9pby9maWxlYmFzZWRzb3VyY2UucHk=) | | |
| [.../apache\_beam/testing/benchmarks/nexmark/monitor.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS90ZXN0aW5nL2JlbmNobWFya3MvbmV4bWFyay9tb25pdG9yLnB5) | | |
| [...cs/sdks/python/apache\_beam/dataframe/frame\_base.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9kYXRhZnJhbWUvZnJhbWVfYmFzZS5weQ==) | | |
| [...s/python/apache\_beam/runners/portability/stager.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL3BvcnRhYmlsaXR5L3N0YWdlci5weQ==) | | |
| ... and [928 more](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=footer). Last update [1f35e2f...9cbc36c](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] codecov[bot] edited a comment on pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
codecov[bot] edited a comment on pull request #14145:
URL: https://github.com/apache/beam/pull/14145#issuecomment-790308850
# [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=h1) Report
> Merging [#14145](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=desc) (04ea4e3) into [master](https://codecov.io/gh/apache/beam/commit/2d9c66647ecbc9aca6a8618168d44e4c815cea19?el=desc) (2d9c666) will **increase** coverage by `0.01%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #14145 +/- ##
==========================================
+ Coverage 83.04% 83.05% +0.01%
==========================================
Files 469 469
Lines 58472 58493 +21
==========================================
+ Hits 48556 48580 +24
+ Misses 9916 9913 -3
```
| [Impacted Files](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [.../runners/interactive/testing/test\_cache\_manager.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL2ludGVyYWN0aXZlL3Rlc3RpbmcvdGVzdF9jYWNoZV9tYW5hZ2VyLnB5) | `91.30% <0.00%> (-2.90%)` | :arrow_down: |
| [...sdks/python/apache\_beam/transforms/environments.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS90cmFuc2Zvcm1zL2Vudmlyb25tZW50cy5weQ==) | `85.24% <0.00%> (+0.04%)` | :arrow_up: |
| [...srcs/sdks/python/apache\_beam/dataframe/doctests.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9kYXRhZnJhbWUvZG9jdGVzdHMucHk=) | `97.74% <0.00%> (+0.05%)` | :arrow_up: |
| [...d/srcs/sdks/python/apache\_beam/dataframe/frames.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9kYXRhZnJhbWUvZnJhbWVzLnB5) | `91.00% <0.00%> (+0.20%)` | :arrow_up: |
| [...eam/runners/interactive/interactive\_environment.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL2ludGVyYWN0aXZlL2ludGVyYWN0aXZlX2Vudmlyb25tZW50LnB5) | `90.28% <0.00%> (+0.35%)` | :arrow_up: |
| [...hon/apache\_beam/runners/direct/test\_stream\_impl.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL2RpcmVjdC90ZXN0X3N0cmVhbV9pbXBsLnB5) | `94.11% <0.00%> (+2.20%)` | :arrow_up: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=footer). Last update [1f35e2f...04ea4e3](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] codecov[bot] edited a comment on pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
codecov[bot] edited a comment on pull request #14145:
URL: https://github.com/apache/beam/pull/14145#issuecomment-790308850
# [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=h1) Report
> Merging [#14145](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=desc) (9cbc36c) into [master](https://codecov.io/gh/apache/beam/commit/2d9c66647ecbc9aca6a8618168d44e4c815cea19?el=desc) (2d9c666) will **increase** coverage by `0.00%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #14145 +/- ##
=======================================
Coverage 83.04% 83.04%
=======================================
Files 469 469
Lines 58472 58493 +21
=======================================
+ Hits 48556 48577 +21
Misses 9916 9916
```
| [Impacted Files](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [.../runners/interactive/testing/test\_cache\_manager.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL2ludGVyYWN0aXZlL3Rlc3RpbmcvdGVzdF9jYWNoZV9tYW5hZ2VyLnB5) | `91.30% <0.00%> (-2.90%)` | :arrow_down: |
| [...sdks/python/apache\_beam/transforms/environments.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS90cmFuc2Zvcm1zL2Vudmlyb25tZW50cy5weQ==) | `85.24% <0.00%> (+0.04%)` | :arrow_up: |
| [...srcs/sdks/python/apache\_beam/dataframe/doctests.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9kYXRhZnJhbWUvZG9jdGVzdHMucHk=) | `97.74% <0.00%> (+0.05%)` | :arrow_up: |
| [...d/srcs/sdks/python/apache\_beam/dataframe/frames.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9kYXRhZnJhbWUvZnJhbWVzLnB5) | `91.00% <0.00%> (+0.20%)` | :arrow_up: |
| [...eam/runners/interactive/interactive\_environment.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL2ludGVyYWN0aXZlL2ludGVyYWN0aXZlX2Vudmlyb25tZW50LnB5) | `90.28% <0.00%> (+0.35%)` | :arrow_up: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=footer). Last update [1f35e2f...9cbc36c](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] tvalentyn commented on pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
tvalentyn commented on pull request #14145:
URL: https://github.com/apache/beam/pull/14145#issuecomment-791082563
Run Seed Job
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] y1chi commented on pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
y1chi commented on pull request #14145:
URL: https://github.com/apache/beam/pull/14145#issuecomment-793051870
Run Python Dataflow ValidatesContainer
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] y1chi removed a comment on pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
y1chi removed a comment on pull request #14145:
URL: https://github.com/apache/beam/pull/14145#issuecomment-790301677
Run Seed Job
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] tvalentyn commented on pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
tvalentyn commented on pull request #14145:
URL: https://github.com/apache/beam/pull/14145#issuecomment-793168499
We would need to re-run the seed job to run the suite again. The last time I ran the seed job (on #13626) and
and then the suite on this PR, I got:
```
13:38:09 Task ':sdks:python:container:buildDarwinAmd64' is not up-to-date because:
13:38:09 No history is available.
13:38:09
13:38:09 .gogradle/project_gopath/src/github.com/apache/beam/sdks/python/boot/boot.go:34:2: cannot find package "github.com/apache/beam/sdks/go/pkg/beam/artifact" in any of:
13:38:09 /home/jenkins/jenkins-slave/workspace/beam_PostCommit_Py_ValCont_PR/src/sdks/python/container/.gogradle/project_gopath/src/github.com/apache/beam/sdks/python/boot/vendor/github.com/apache/beam/sdks/go/pkg/beam/artifact (vendor tree)
13:38:09 /home/jenkins/.gradle/go/binary/1.12.7/go/src/github.com/apache/beam/sdks/go/pkg/beam/artifact (from $GOROOT)
13:38:09 /home/jenkins/jenkins-slave/workspace/beam_PostCommit_Py_ValCont_PR/src/sdks/python/container/.gogradle/project_gopath/src/github.com/apache/beam/sdks/go/pkg/beam/artifact (from $GOPATH)
13:38:09 .gogradle/project_gopath/src/github.com/apache/beam/sdks/python/boot/boot.go:35:2: cannot find package "github.com/apache/beam/sdks/go/pkg/beam/model/pipeline_v1" in any of:
13:38:09 /home/jenkins/jenkins-slave/workspace/beam_PostCommit_Py_ValCont_PR/src/sdks/python/container/.gogradle/project_gopath/src/github.com/apache/beam/sdks/python/boot/vendor/github.com/apache/beam/sdks/go/pkg/beam/model/pipeline_v1 (vendor tree)
13:38:09 /home/jenkins/.gradle/go/binary/1.12.7/go/src/github.com/apache/beam/sdks/go/pkg/beam/model/pipeline_v1 (from $GOROOT)
13:38:09
13:38:09 /home/jenkins/jenkins-slave/workspace/beam_PostCommit_Py_ValCont_PR/src/sdks/python/container/.gogradle/project_gopath/src/github.com/apache/beam/sdks/go/pkg/beam/model/pipeline_v1 (from $GOPATH)
13:38:09 > Task :sdks:python:container:buildDarwinAmd64 FAILED
13:38:09 .gogradle/project_gopath/src/github.com/apache/beam/sdks/python/boot/boot.go:36:2: cannot find package "github.com/apache/beam/sdks/go/pkg/beam/provision" in any of:
13:38:09 :sdks:python:container:buildDarwinAmd64 (Thread[Execution worker for ':' Thread 4,5,main]) completed. Took 3.908 secs.
13:38:09 /home/jenkins/jenkins-slave/workspace/beam_PostCommit_Py_ValCont_PR/src/sdks/python/container/.gogradle/project_gopath/src/github.com/apache/beam/sdks/python/boot/vendor/github.com/apache/beam/sdks/go/pkg/beam/provision (vendor tree)
13:38:09 /home/jenkins/.gradle/go/binary/1.12.7/go/src/github.com/apache/beam/sdks/go/pkg/beam/provision (from $GOROOT)
13:38:09 /home/jenkins/jenkins-slave/workspace/beam_PostCommit_Py_ValCont_PR/src/sdks/python/container/.gogradle/project_gopath/src/github.com/apache/beam/sdks/go/pkg/beam/provision (from $GOPATH)
13:38:09 .gogradle/project_gopath/src/github.com/apache/beam/sdks/python/boot/boot.go:37:2: cannot find package "github.com/apache/beam/sdks/go/pkg/beam/util/execx" in any of:
13:38:09 /home/jenkins/jenkins-slave/workspace/beam_PostCommit_Py_ValCont_PR/src/sdks/python/container/.gogradle/project_gopath/src/github.com/apache/beam/sdks/python/boot/vendor/github.com/apache/beam/sdks/go/pkg/beam/util/execx (vendor tree)
13:38:09 /home/jenkins/.gradle/go/binary/1.12.7/go/src/github.com/apache/beam/sdks/go/pkg/beam/util/execx (from $GOROOT)
13:38:09 /home/jenkins/jenkins-slave/workspace/beam_PostCommit_Py_ValCont_PR/src/sdks/python/container/.gogradle/project_gopath/src/github.com/apache/beam/sdks/go/pkg/beam/util/execx (from $GOPATH)
13:38:09 .gogradle/project_gopath/src/github.com/apache/beam/sdks/python/boot/boot.go:38:2: cannot find package "github.com/apache/beam/sdks/go/pkg/beam/util/grpcx" in any of:
13:38:09 /home/jenkins/jenkins-slave/workspace/beam_PostCommit_Py_ValCont_PR/src/sdks/python/container/.gogradle/project_gopath/src/github.com/apache/beam/sdks/python/boot/vendor/github.com/apache/beam/sdks/go/pkg/beam/util/grpcx (vendor tree)
13:38:09 /home/jenkins/.gradle/go/binary/1.12.7/go/src/github.com/apache/beam/sdks/go/pkg/beam/util/grpcx (from $GOROOT)
13:38:09 /home/jenkins/jenkins-slave/workspace/beam_PostCommit_Py_ValCont_PR/src/sdks/python/container/.gogradle/project_gopath/src/github.com/apache/beam/sdks/go/pkg/beam/util/grpcx (from $GOPATH)
13:38:09 .gogradle/project_gopath/src/github.com/apache/beam/sdks/python/boot/boot.go:39:2: cannot find package "github.com/golang/protobuf/jsonpb" in any of:
13:38:09 /home/jenkins/jenkins-slave/workspace/beam_PostCommit_Py_ValCont_PR/src/sdks/python/container/.gogradle/project_gopath/src/github.com/apache/beam/sdks/python/boot/vendor/github.com/golang/protobuf/jsonpb (vendor tree)
13:38:09 /home/jenkins/.gradle/go/binary/1.12.7/go/src/github.com/golang/protobuf/jsonpb (from $GOROOT)
13:38:09 /home/jenkins/jenkins-slave/workspace/beam_PostCommit_Py_ValCont_PR/src/sdks/python/container/.gogradle/project_gopath/src/github.com/golang/protobuf/jsonpb (from $GOPATH)
13:38:09 .gogradle/project_gopath/src/github.com/apache/beam/sdks/python/boot/boot.go:40:2: cannot find package "github.com/golang/protobuf/proto" in any of:
13:38:09 /home/jenkins/jenkins-slave/workspace/beam_PostCommit_Py_ValCont_PR/src/sdks/python/container/.gogradle/project_gopath/src/github.com/apache/beam/sdks/python/boot/vendor/github.com/golang/protobuf/proto (vendor tree)
13:38:09 /home/jenkins/.gradle/go/binary/1.12.7/go/src/github.com/golang/protobuf/proto (from $GOROOT)
13:38:09 /home/jenkins/jenkins-slave/workspace/beam_PostCommit_Py_ValCont_PR/src/sdks/python/container/.gogradle/project_gopath/src/github.com/golang/protobuf/proto (from $GOPATH)
13:38:09
13:38:09 FAILURE: Build failed with an exception.
13:38:09
13:38:09 * What went wrong:
13:38:09 Execution failed for task ':sdks:python:container:buildDarwinAmd64'.
13:38:09 > Build failed due to return code 1 of:
13:38:09 Command:
13:38:09 /home/jenkins/.gradle/go/binary/1.12.7/go/bin/go build -o ./build/target/launcher/darwin_amd64/boot github.com/apache/beam/sdks/python/boot
13:38:09 Env:
13:38:09
```
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] codecov[bot] edited a comment on pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
codecov[bot] edited a comment on pull request #14145:
URL: https://github.com/apache/beam/pull/14145#issuecomment-790308850
# [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=h1) Report
> Merging [#14145](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=desc) (9cbc36c) into [master](https://codecov.io/gh/apache/beam/commit/2d9c66647ecbc9aca6a8618168d44e4c815cea19?el=desc) (2d9c666) will **increase** coverage by `0.00%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #14145 +/- ##
=======================================
Coverage 83.04% 83.04%
=======================================
Files 469 469
Lines 58472 58493 +21
=======================================
+ Hits 48556 48577 +21
Misses 9916 9916
```
| [Impacted Files](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [.../runners/interactive/testing/test\_cache\_manager.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL2ludGVyYWN0aXZlL3Rlc3RpbmcvdGVzdF9jYWNoZV9tYW5hZ2VyLnB5) | `91.30% <0.00%> (-2.90%)` | :arrow_down: |
| [...sdks/python/apache\_beam/transforms/environments.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS90cmFuc2Zvcm1zL2Vudmlyb25tZW50cy5weQ==) | `85.24% <0.00%> (+0.04%)` | :arrow_up: |
| [...srcs/sdks/python/apache\_beam/dataframe/doctests.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9kYXRhZnJhbWUvZG9jdGVzdHMucHk=) | `97.74% <0.00%> (+0.05%)` | :arrow_up: |
| [...d/srcs/sdks/python/apache\_beam/dataframe/frames.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9kYXRhZnJhbWUvZnJhbWVzLnB5) | `91.00% <0.00%> (+0.20%)` | :arrow_up: |
| [...eam/runners/interactive/interactive\_environment.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL2ludGVyYWN0aXZlL2ludGVyYWN0aXZlX2Vudmlyb25tZW50LnB5) | `90.28% <0.00%> (+0.35%)` | :arrow_up: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=footer). Last update [1f35e2f...9cbc36c](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] codecov[bot] edited a comment on pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
codecov[bot] edited a comment on pull request #14145:
URL: https://github.com/apache/beam/pull/14145#issuecomment-790308850
# [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=h1) Report
> Merging [#14145](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=desc) (9cbc36c) into [master](https://codecov.io/gh/apache/beam/commit/2d9c66647ecbc9aca6a8618168d44e4c815cea19?el=desc) (2d9c666) will **increase** coverage by `0.00%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #14145 +/- ##
=======================================
Coverage 83.04% 83.04%
=======================================
Files 469 469
Lines 58472 58493 +21
=======================================
+ Hits 48556 48577 +21
Misses 9916 9916
```
| [Impacted Files](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [.../runners/interactive/testing/test\_cache\_manager.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL2ludGVyYWN0aXZlL3Rlc3RpbmcvdGVzdF9jYWNoZV9tYW5hZ2VyLnB5) | `91.30% <0.00%> (-2.90%)` | :arrow_down: |
| [...sdks/python/apache\_beam/transforms/environments.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS90cmFuc2Zvcm1zL2Vudmlyb25tZW50cy5weQ==) | `85.24% <0.00%> (+0.04%)` | :arrow_up: |
| [...srcs/sdks/python/apache\_beam/dataframe/doctests.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9kYXRhZnJhbWUvZG9jdGVzdHMucHk=) | `97.74% <0.00%> (+0.05%)` | :arrow_up: |
| [...d/srcs/sdks/python/apache\_beam/dataframe/frames.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9kYXRhZnJhbWUvZnJhbWVzLnB5) | `91.00% <0.00%> (+0.20%)` | :arrow_up: |
| [...eam/runners/interactive/interactive\_environment.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL2ludGVyYWN0aXZlL2ludGVyYWN0aXZlX2Vudmlyb25tZW50LnB5) | `90.28% <0.00%> (+0.35%)` | :arrow_up: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=footer). Last update [1f35e2f...9cbc36c](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] tvalentyn removed a comment on pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
tvalentyn removed a comment on pull request #14145:
URL: https://github.com/apache/beam/pull/14145#issuecomment-796442584
Run Seed Job
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] tvalentyn commented on pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
tvalentyn commented on pull request #14145:
URL: https://github.com/apache/beam/pull/14145#issuecomment-796442584
Run Seed Job
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] codecov[bot] edited a comment on pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
codecov[bot] edited a comment on pull request #14145:
URL: https://github.com/apache/beam/pull/14145#issuecomment-790308850
# [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=h1) Report
> Merging [#14145](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=desc) (a59293d) into [master](https://codecov.io/gh/apache/beam/commit/2d9c66647ecbc9aca6a8618168d44e4c815cea19?el=desc) (2d9c666) will **increase** coverage by `0.00%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #14145 +/- ##
=======================================
Coverage 83.04% 83.05%
=======================================
Files 469 469
Lines 58472 58493 +21
=======================================
+ Hits 48556 48579 +23
+ Misses 9916 9914 -2
```
| [Impacted Files](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [...s/sdks/python/apache\_beam/io/gcp/tests/\_\_init\_\_.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9pby9nY3AvdGVzdHMvX19pbml0X18ucHk=) | | |
| [...apache\_beam/examples/cookbook/custom\_ptransform.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9leGFtcGxlcy9jb29rYm9vay9jdXN0b21fcHRyYW5zZm9ybS5weQ==) | | |
| [...che\_beam/examples/streaming\_wordcount\_debugging.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9leGFtcGxlcy9zdHJlYW1pbmdfd29yZGNvdW50X2RlYnVnZ2luZy5weQ==) | | |
| [...ld/srcs/sdks/python/apache\_beam/io/gcp/bigquery.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9pby9nY3AvYmlncXVlcnkucHk=) | | |
| [...e\_beam/runners/portability/abstract\_job\_service.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL3BvcnRhYmlsaXR5L2Fic3RyYWN0X2pvYl9zZXJ2aWNlLnB5) | | |
| [...s/python/apache\_beam/typehints/sharded\_key\_type.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS90eXBlaGludHMvc2hhcmRlZF9rZXlfdHlwZS5weQ==) | | |
| [...srcs/sdks/python/apache\_beam/io/filebasedsource.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9pby9maWxlYmFzZWRzb3VyY2UucHk=) | | |
| [.../apache\_beam/testing/benchmarks/nexmark/monitor.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS90ZXN0aW5nL2JlbmNobWFya3MvbmV4bWFyay9tb25pdG9yLnB5) | | |
| [...cs/sdks/python/apache\_beam/dataframe/frame\_base.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9kYXRhZnJhbWUvZnJhbWVfYmFzZS5weQ==) | | |
| [...s/python/apache\_beam/runners/portability/stager.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL3BvcnRhYmlsaXR5L3N0YWdlci5weQ==) | | |
| ... and [928 more](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=footer). Last update [1f35e2f...a59293d](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] angoenka commented on a change in pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
angoenka commented on a change in pull request #14145:
URL: https://github.com/apache/beam/pull/14145#discussion_r587662530
##########
File path: .test-infra/jenkins/job_PostCommit_Python_ValidatesContainer_Dataflow.groovy
##########
@@ -35,11 +35,13 @@ PostcommitJobBuilder.postCommitJob('beam_PostCommit_Py_ValCont',
}
// Execute shell command to test Python SDK.
- // TODO: Parallel the script run with Jenkins DSL or Gradle.
steps {
- VALIDATES_CONTAINER_DATAFLOW_PYTHON_VERSIONS.each {
- pythonVersion = it.replace('.', '')
- shell('cd ' + commonJobProperties.checkoutDir + " && bash sdks/python/container/run_validatescontainer.sh python${pythonVersion}")
+ gradle {
+ rootBuildScriptDir(commonJobProperties.checkoutDir)
+ tasks(':sdks:python:test-suites:portable:py36:validatesContainer')
Review comment:
This list will need to be updated with every new Python version.
Shall we update the relevant doc for adding and removing python versions.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] codecov[bot] edited a comment on pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
codecov[bot] edited a comment on pull request #14145:
URL: https://github.com/apache/beam/pull/14145#issuecomment-790308850
# [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=h1) Report
> Merging [#14145](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=desc) (15e2d0b) into [master](https://codecov.io/gh/apache/beam/commit/2d9c66647ecbc9aca6a8618168d44e4c815cea19?el=desc) (2d9c666) will **decrease** coverage by `0.00%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #14145 +/- ##
==========================================
- Coverage 83.04% 83.04% -0.01%
==========================================
Files 469 469
Lines 58472 58473 +1
==========================================
Hits 48556 48556
- Misses 9916 9917 +1
```
| [Impacted Files](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [...38/build/srcs/sdks/python/apache\_beam/io/iobase.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9pby9pb2Jhc2UucHk=) | `84.55% <0.00%> (-0.27%)` | :arrow_down: |
| [...sdks/python/apache\_beam/transforms/environments.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS90cmFuc2Zvcm1zL2Vudmlyb25tZW50cy5weQ==) | `85.24% <0.00%> (+0.04%)` | :arrow_up: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=footer). Last update [1f35e2f...15e2d0b](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] codecov[bot] edited a comment on pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
codecov[bot] edited a comment on pull request #14145:
URL: https://github.com/apache/beam/pull/14145#issuecomment-790308850
# [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=h1) Report
> Merging [#14145](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=desc) (b473163) into [master](https://codecov.io/gh/apache/beam/commit/2d9c66647ecbc9aca6a8618168d44e4c815cea19?el=desc) (2d9c666) will **increase** coverage by `0.12%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #14145 +/- ##
==========================================
+ Coverage 83.04% 83.16% +0.12%
==========================================
Files 469 469
Lines 58472 58954 +482
==========================================
+ Hits 48556 49029 +473
- Misses 9916 9925 +9
```
| [Impacted Files](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [.../runners/interactive/testing/test\_cache\_manager.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL2ludGVyYWN0aXZlL3Rlc3RpbmcvdGVzdF9jYWNoZV9tYW5hZ2VyLnB5) | `91.30% <0.00%> (-2.90%)` | :arrow_down: |
| [.../python/apache\_beam/transforms/periodicsequence.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS90cmFuc2Zvcm1zL3BlcmlvZGljc2VxdWVuY2UucHk=) | `96.49% <0.00%> (-1.76%)` | :arrow_down: |
| [.../internal/clients/dataflow/dataflow\_v1b3\_client.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL2RhdGFmbG93L2ludGVybmFsL2NsaWVudHMvZGF0YWZsb3cvZGF0YWZsb3dfdjFiM19jbGllbnQucHk=) | `71.34% <0.00%> (-1.47%)` | :arrow_down: |
| [...cs/sdks/python/apache\_beam/dataframe/frame\_base.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9kYXRhZnJhbWUvZnJhbWVfYmFzZS5weQ==) | `89.70% <0.00%> (-0.80%)` | :arrow_down: |
| [...on/apache\_beam/runners/portability/spark\_runner.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL3BvcnRhYmlsaXR5L3NwYXJrX3J1bm5lci5weQ==) | `55.81% <0.00%> (-0.71%)` | :arrow_down: |
| [.../srcs/sdks/python/apache\_beam/dataframe/convert.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9kYXRhZnJhbWUvY29udmVydC5weQ==) | `90.47% <0.00%> (-0.44%)` | :arrow_down: |
| [...on/apache\_beam/runners/portability/flink\_runner.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL3BvcnRhYmlsaXR5L2ZsaW5rX3J1bm5lci5weQ==) | `61.11% <0.00%> (-0.30%)` | :arrow_down: |
| [...srcs/sdks/python/apache\_beam/dataframe/doctests.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9kYXRhZnJhbWUvZG9jdGVzdHMucHk=) | `97.55% <0.00%> (-0.15%)` | :arrow_down: |
| [...ks/python/apache\_beam/runners/worker/operations.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL3dvcmtlci9vcGVyYXRpb25zLnB5) | `70.27% <0.00%> (-0.11%)` | :arrow_down: |
| [...on/apache\_beam/runners/dataflow/dataflow\_runner.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL2RhdGFmbG93L2RhdGFmbG93X3J1bm5lci5weQ==) | `81.83% <0.00%> (-0.07%)` | :arrow_down: |
| ... and [23 more](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=footer). Last update [1f35e2f...b473163](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] codecov[bot] edited a comment on pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
codecov[bot] edited a comment on pull request #14145:
URL: https://github.com/apache/beam/pull/14145#issuecomment-790308850
# [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=h1) Report
> Merging [#14145](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=desc) (15e2d0b) into [master](https://codecov.io/gh/apache/beam/commit/2d9c66647ecbc9aca6a8618168d44e4c815cea19?el=desc) (2d9c666) will **decrease** coverage by `0.00%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #14145 +/- ##
==========================================
- Coverage 83.04% 83.04% -0.01%
==========================================
Files 469 469
Lines 58472 58473 +1
==========================================
Hits 48556 48556
- Misses 9916 9917 +1
```
| [Impacted Files](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [...38/build/srcs/sdks/python/apache\_beam/io/iobase.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9pby9pb2Jhc2UucHk=) | `84.55% <0.00%> (-0.27%)` | :arrow_down: |
| [...sdks/python/apache\_beam/transforms/environments.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS90cmFuc2Zvcm1zL2Vudmlyb25tZW50cy5weQ==) | `85.24% <0.00%> (+0.04%)` | :arrow_up: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=footer). Last update [1f35e2f...15e2d0b](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] tvalentyn commented on pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
tvalentyn commented on pull request #14145:
URL: https://github.com/apache/beam/pull/14145#issuecomment-791099215
```
18:09:37 Running setup.py (path:/tmp/pip-req-build-kxmdxkxz/setup.py) egg_info for package from file:///opt/apache/beam/tars/apache-beam.tar.gz
18:09:37 Created temporary directory: /tmp/pip-pip-egg-info-cw3flwuy
18:09:37 Running command python setup.py egg_info
18:09:38 Traceback (most recent call last):
18:09:38 File "<string>", line 1, in <module>
18:09:38 File "/usr/local/lib/python3.6/tokenize.py", line 452, in open
18:09:38 buffer = _builtin_open(filename, 'rb')
18:09:38 FileNotFoundError: [Errno 2] No such file or directory: '/tmp/pip-req-build-kxmdxkxz/setup.py'
18:09:38 WARNING: Discarding file:///opt/apache/beam/tars/apache-beam.tar.gz. Command errored out with exit status 1: python setup.py egg_info Check the logs for full command output.
18:09:38 ERROR: Command errored out with exit status 1: python setup.py egg_info Check the logs for full command output.
```
Looks like some bits of the shell script may have parallelization issues. I vaguely recall having issues with either creating virtualenv or in parallel, and or building sdk tarball sdist in parallel, which we solved by configuring dependency chains in common python tasks.
Adding
```
dependsOn 'installGcpTest'
dependsOn ':sdks:python:sdist'
```
Instead of https://github.com/y1chi/beam/blob/python_valcon/sdks/python/container/run_validatescontainer.sh#L109-L118
might resolve it.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] y1chi commented on pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
y1chi commented on pull request #14145:
URL: https://github.com/apache/beam/pull/14145#issuecomment-790301677
Run Seed Job
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] codecov[bot] edited a comment on pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
codecov[bot] edited a comment on pull request #14145:
URL: https://github.com/apache/beam/pull/14145#issuecomment-790308850
# [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=h1) Report
> Merging [#14145](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=desc) (b473163) into [master](https://codecov.io/gh/apache/beam/commit/2d9c66647ecbc9aca6a8618168d44e4c815cea19?el=desc) (2d9c666) will **increase** coverage by `0.12%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #14145 +/- ##
==========================================
+ Coverage 83.04% 83.16% +0.12%
==========================================
Files 469 469
Lines 58472 58954 +482
==========================================
+ Hits 48556 49029 +473
- Misses 9916 9925 +9
```
| [Impacted Files](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [.../runners/interactive/testing/test\_cache\_manager.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL2ludGVyYWN0aXZlL3Rlc3RpbmcvdGVzdF9jYWNoZV9tYW5hZ2VyLnB5) | `91.30% <0.00%> (-2.90%)` | :arrow_down: |
| [.../python/apache\_beam/transforms/periodicsequence.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS90cmFuc2Zvcm1zL3BlcmlvZGljc2VxdWVuY2UucHk=) | `96.49% <0.00%> (-1.76%)` | :arrow_down: |
| [.../internal/clients/dataflow/dataflow\_v1b3\_client.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL2RhdGFmbG93L2ludGVybmFsL2NsaWVudHMvZGF0YWZsb3cvZGF0YWZsb3dfdjFiM19jbGllbnQucHk=) | `71.34% <0.00%> (-1.47%)` | :arrow_down: |
| [...cs/sdks/python/apache\_beam/dataframe/frame\_base.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9kYXRhZnJhbWUvZnJhbWVfYmFzZS5weQ==) | `89.70% <0.00%> (-0.80%)` | :arrow_down: |
| [...on/apache\_beam/runners/portability/spark\_runner.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL3BvcnRhYmlsaXR5L3NwYXJrX3J1bm5lci5weQ==) | `55.81% <0.00%> (-0.71%)` | :arrow_down: |
| [.../srcs/sdks/python/apache\_beam/dataframe/convert.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9kYXRhZnJhbWUvY29udmVydC5weQ==) | `90.47% <0.00%> (-0.44%)` | :arrow_down: |
| [...on/apache\_beam/runners/portability/flink\_runner.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL3BvcnRhYmlsaXR5L2ZsaW5rX3J1bm5lci5weQ==) | `61.11% <0.00%> (-0.30%)` | :arrow_down: |
| [...srcs/sdks/python/apache\_beam/dataframe/doctests.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9kYXRhZnJhbWUvZG9jdGVzdHMucHk=) | `97.55% <0.00%> (-0.15%)` | :arrow_down: |
| [...ks/python/apache\_beam/runners/worker/operations.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL3dvcmtlci9vcGVyYXRpb25zLnB5) | `70.27% <0.00%> (-0.11%)` | :arrow_down: |
| [...on/apache\_beam/runners/dataflow/dataflow\_runner.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL2RhdGFmbG93L2RhdGFmbG93X3J1bm5lci5weQ==) | `81.83% <0.00%> (-0.07%)` | :arrow_down: |
| ... and [23 more](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=footer). Last update [1f35e2f...b473163](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] codecov[bot] edited a comment on pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
codecov[bot] edited a comment on pull request #14145:
URL: https://github.com/apache/beam/pull/14145#issuecomment-790308850
# [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=h1) Report
> Merging [#14145](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=desc) (04ea4e3) into [master](https://codecov.io/gh/apache/beam/commit/2d9c66647ecbc9aca6a8618168d44e4c815cea19?el=desc) (2d9c666) will **increase** coverage by `0.01%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #14145 +/- ##
==========================================
+ Coverage 83.04% 83.05% +0.01%
==========================================
Files 469 469
Lines 58472 58493 +21
==========================================
+ Hits 48556 48580 +24
+ Misses 9916 9913 -3
```
| [Impacted Files](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [.../runners/interactive/testing/test\_cache\_manager.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL2ludGVyYWN0aXZlL3Rlc3RpbmcvdGVzdF9jYWNoZV9tYW5hZ2VyLnB5) | `91.30% <0.00%> (-2.90%)` | :arrow_down: |
| [...sdks/python/apache\_beam/transforms/environments.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS90cmFuc2Zvcm1zL2Vudmlyb25tZW50cy5weQ==) | `85.24% <0.00%> (+0.04%)` | :arrow_up: |
| [...srcs/sdks/python/apache\_beam/dataframe/doctests.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9kYXRhZnJhbWUvZG9jdGVzdHMucHk=) | `97.74% <0.00%> (+0.05%)` | :arrow_up: |
| [...d/srcs/sdks/python/apache\_beam/dataframe/frames.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9kYXRhZnJhbWUvZnJhbWVzLnB5) | `91.00% <0.00%> (+0.20%)` | :arrow_up: |
| [...eam/runners/interactive/interactive\_environment.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL2ludGVyYWN0aXZlL2ludGVyYWN0aXZlX2Vudmlyb25tZW50LnB5) | `90.28% <0.00%> (+0.35%)` | :arrow_up: |
| [...hon/apache\_beam/runners/direct/test\_stream\_impl.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL2RpcmVjdC90ZXN0X3N0cmVhbV9pbXBsLnB5) | `94.11% <0.00%> (+2.20%)` | :arrow_up: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=footer). Last update [1f35e2f...04ea4e3](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] codecov[bot] edited a comment on pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
codecov[bot] edited a comment on pull request #14145:
URL: https://github.com/apache/beam/pull/14145#issuecomment-790308850
# [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=h1) Report
> Merging [#14145](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=desc) (2b8938e) into [master](https://codecov.io/gh/apache/beam/commit/2d9c66647ecbc9aca6a8618168d44e4c815cea19?el=desc) (2d9c666) will **increase** coverage by `0.01%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #14145 +/- ##
==========================================
+ Coverage 83.04% 83.05% +0.01%
==========================================
Files 469 469
Lines 58472 58473 +1
==========================================
+ Hits 48556 48563 +7
+ Misses 9916 9910 -6
```
| [Impacted Files](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [...s/python/apache\_beam/examples/complete/\_\_init\_\_.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9leGFtcGxlcy9jb21wbGV0ZS9fX2luaXRfXy5weQ==) | | |
| [.../srcs/sdks/python/apache\_beam/coders/typecoders.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9jb2RlcnMvdHlwZWNvZGVycy5weQ==) | | |
| [.../python/apache\_beam/io/gcp/tests/pubsub\_matcher.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9pby9nY3AvdGVzdHMvcHVic3ViX21hdGNoZXIucHk=) | | |
| [...beam/runners/portability/fn\_api\_runner/\_\_init\_\_.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL3BvcnRhYmlsaXR5L2ZuX2FwaV9ydW5uZXIvX19pbml0X18ucHk=) | | |
| [...ld/srcs/sdks/python/apache\_beam/coders/\_\_init\_\_.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9jb2RlcnMvX19pbml0X18ucHk=) | | |
| [...thon/apache\_beam/examples/complete/autocomplete.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9leGFtcGxlcy9jb21wbGV0ZS9hdXRvY29tcGxldGUucHk=) | | |
| [...\_beam/testing/benchmarks/nexmark/queries/query7.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS90ZXN0aW5nL2JlbmNobWFya3MvbmV4bWFyay9xdWVyaWVzL3F1ZXJ5Ny5weQ==) | | |
| [.../srcs/sdks/python/apache\_beam/typehints/schemas.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS90eXBlaGludHMvc2NoZW1hcy5weQ==) | | |
| [.../sdks/python/apache\_beam/internal/metrics/cells.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9pbnRlcm5hbC9tZXRyaWNzL2NlbGxzLnB5) | | |
| [...s/python/apache\_beam/io/aws/clients/s3/\_\_init\_\_.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9pby9hd3MvY2xpZW50cy9zMy9fX2luaXRfXy5weQ==) | | |
| ... and [928 more](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=footer). Last update [1f35e2f...15e2d0b](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] tvalentyn merged pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
tvalentyn merged pull request #14145:
URL: https://github.com/apache/beam/pull/14145
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] angoenka commented on pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
angoenka commented on pull request #14145:
URL: https://github.com/apache/beam/pull/14145#issuecomment-790782335
cc @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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] codecov[bot] edited a comment on pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
codecov[bot] edited a comment on pull request #14145:
URL: https://github.com/apache/beam/pull/14145#issuecomment-790308850
# [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=h1) Report
> Merging [#14145](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=desc) (9cbc36c) into [master](https://codecov.io/gh/apache/beam/commit/2d9c66647ecbc9aca6a8618168d44e4c815cea19?el=desc) (2d9c666) will **increase** coverage by `0.00%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #14145 +/- ##
=======================================
Coverage 83.04% 83.04%
=======================================
Files 469 469
Lines 58472 58493 +21
=======================================
+ Hits 48556 48577 +21
Misses 9916 9916
```
| [Impacted Files](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [.../runners/interactive/testing/test\_cache\_manager.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL2ludGVyYWN0aXZlL3Rlc3RpbmcvdGVzdF9jYWNoZV9tYW5hZ2VyLnB5) | `91.30% <0.00%> (-2.90%)` | :arrow_down: |
| [...sdks/python/apache\_beam/transforms/environments.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS90cmFuc2Zvcm1zL2Vudmlyb25tZW50cy5weQ==) | `85.24% <0.00%> (+0.04%)` | :arrow_up: |
| [...srcs/sdks/python/apache\_beam/dataframe/doctests.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9kYXRhZnJhbWUvZG9jdGVzdHMucHk=) | `97.74% <0.00%> (+0.05%)` | :arrow_up: |
| [...d/srcs/sdks/python/apache\_beam/dataframe/frames.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9kYXRhZnJhbWUvZnJhbWVzLnB5) | `91.00% <0.00%> (+0.20%)` | :arrow_up: |
| [...eam/runners/interactive/interactive\_environment.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL2ludGVyYWN0aXZlL2ludGVyYWN0aXZlX2Vudmlyb25tZW50LnB5) | `90.28% <0.00%> (+0.35%)` | :arrow_up: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=footer). Last update [1f35e2f...9cbc36c](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] tvalentyn commented on pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
tvalentyn commented on pull request #14145:
URL: https://github.com/apache/beam/pull/14145#issuecomment-796447440
Run Python Dataflow ValidatesContainer
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] codecov[bot] edited a comment on pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
codecov[bot] edited a comment on pull request #14145:
URL: https://github.com/apache/beam/pull/14145#issuecomment-790308850
# [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=h1) Report
> Merging [#14145](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=desc) (a59293d) into [master](https://codecov.io/gh/apache/beam/commit/2d9c66647ecbc9aca6a8618168d44e4c815cea19?el=desc) (2d9c666) will **increase** coverage by `0.00%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #14145 +/- ##
=======================================
Coverage 83.04% 83.05%
=======================================
Files 469 469
Lines 58472 58493 +21
=======================================
+ Hits 48556 48579 +23
+ Misses 9916 9914 -2
```
| [Impacted Files](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [...dks/python/apache\_beam/options/pipeline\_options.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9vcHRpb25zL3BpcGVsaW5lX29wdGlvbnMucHk=) | | |
| [...python/apache\_beam/examples/streaming\_wordcount.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9leGFtcGxlcy9zdHJlYW1pbmdfd29yZGNvdW50LnB5) | | |
| [.../examples/snippets/transforms/aggregation/count.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9leGFtcGxlcy9zbmlwcGV0cy90cmFuc2Zvcm1zL2FnZ3JlZ2F0aW9uL2NvdW50LnB5) | | |
| [.../sdks/python/apache\_beam/transforms/deduplicate.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS90cmFuc2Zvcm1zL2RlZHVwbGljYXRlLnB5) | | |
| [.../sdks/python/apache\_beam/examples/snippets/util.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9leGFtcGxlcy9zbmlwcGV0cy91dGlsLnB5) | | |
| [...sdks/python/apache\_beam/examples/flink/\_\_init\_\_.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9leGFtcGxlcy9mbGluay9fX2luaXRfXy5weQ==) | | |
| [...he\_beam/portability/api/external\_transforms\_pb2.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9wb3J0YWJpbGl0eS9hcGkvZXh0ZXJuYWxfdHJhbnNmb3Jtc19wYjIucHk=) | | |
| [...sdks/python/apache\_beam/utils/subprocess\_server.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS91dGlscy9zdWJwcm9jZXNzX3NlcnZlci5weQ==) | | |
| [...thon/apache\_beam/runners/worker/operation\_specs.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL3dvcmtlci9vcGVyYXRpb25fc3BlY3MucHk=) | | |
| [...e\_beam/portability/api/beam\_runner\_api\_pb2\_urns.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9wb3J0YWJpbGl0eS9hcGkvYmVhbV9ydW5uZXJfYXBpX3BiMl91cm5zLnB5) | | |
| ... and [928 more](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=footer). Last update [1f35e2f...a59293d](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] codecov[bot] edited a comment on pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
codecov[bot] edited a comment on pull request #14145:
URL: https://github.com/apache/beam/pull/14145#issuecomment-790308850
# [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=h1) Report
> Merging [#14145](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=desc) (15e2d0b) into [master](https://codecov.io/gh/apache/beam/commit/2d9c66647ecbc9aca6a8618168d44e4c815cea19?el=desc) (2d9c666) will **decrease** coverage by `0.00%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #14145 +/- ##
==========================================
- Coverage 83.04% 83.04% -0.01%
==========================================
Files 469 469
Lines 58472 58473 +1
==========================================
Hits 48556 48556
- Misses 9916 9917 +1
```
| [Impacted Files](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [...38/build/srcs/sdks/python/apache\_beam/io/iobase.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9pby9pb2Jhc2UucHk=) | `84.55% <0.00%> (-0.27%)` | :arrow_down: |
| [...sdks/python/apache\_beam/transforms/environments.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS90cmFuc2Zvcm1zL2Vudmlyb25tZW50cy5weQ==) | `85.24% <0.00%> (+0.04%)` | :arrow_up: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=footer). Last update [1f35e2f...15e2d0b](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] codecov[bot] edited a comment on pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
codecov[bot] edited a comment on pull request #14145:
URL: https://github.com/apache/beam/pull/14145#issuecomment-790308850
# [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=h1) Report
> Merging [#14145](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=desc) (2b8938e) into [master](https://codecov.io/gh/apache/beam/commit/2d9c66647ecbc9aca6a8618168d44e4c815cea19?el=desc) (2d9c666) will **increase** coverage by `0.01%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #14145 +/- ##
==========================================
+ Coverage 83.04% 83.05% +0.01%
==========================================
Files 469 469
Lines 58472 58473 +1
==========================================
+ Hits 48556 48563 +7
+ Misses 9916 9910 -6
```
| [Impacted Files](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [...ive/messaging/interactive\_environment\_inspector.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL2ludGVyYWN0aXZlL21lc3NhZ2luZy9pbnRlcmFjdGl2ZV9lbnZpcm9ubWVudF9pbnNwZWN0b3IucHk=) | | |
| [...cs/sdks/python/apache\_beam/dataframe/frame\_base.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9kYXRhZnJhbWUvZnJhbWVfYmFzZS5weQ==) | | |
| [...rcs/sdks/python/apache\_beam/transforms/external.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS90cmFuc2Zvcm1zL2V4dGVybmFsLnB5) | | |
| [...\_beam/testing/benchmarks/nexmark/queries/query9.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS90ZXN0aW5nL2JlbmNobWFya3MvbmV4bWFyay9xdWVyaWVzL3F1ZXJ5OS5weQ==) | | |
| [...am/testing/benchmarks/chicago\_taxi/trainer/taxi.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS90ZXN0aW5nL2JlbmNobWFya3MvY2hpY2Fnb190YXhpL3RyYWluZXIvdGF4aS5weQ==) | | |
| [...sdks/python/apache\_beam/io/restriction\_trackers.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9pby9yZXN0cmljdGlvbl90cmFja2Vycy5weQ==) | | |
| [...e\_beam/examples/complete/top\_wikipedia\_sessions.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9leGFtcGxlcy9jb21wbGV0ZS90b3Bfd2lraXBlZGlhX3Nlc3Npb25zLnB5) | | |
| [...che\_beam/examples/streaming\_wordcount\_debugging.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9leGFtcGxlcy9zdHJlYW1pbmdfd29yZGNvdW50X2RlYnVnZ2luZy5weQ==) | | |
| [...py38/build/srcs/sdks/python/apache\_beam/io/jdbc.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9pby9qZGJjLnB5) | | |
| [...flow/internal/clients/dataflow/message\_matchers.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL2RhdGFmbG93L2ludGVybmFsL2NsaWVudHMvZGF0YWZsb3cvbWVzc2FnZV9tYXRjaGVycy5weQ==) | | |
| ... and [928 more](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=footer). Last update [1f35e2f...a59293d](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] y1chi commented on a change in pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
y1chi commented on a change in pull request #14145:
URL: https://github.com/apache/beam/pull/14145#discussion_r587968823
##########
File path: .test-infra/jenkins/job_PostCommit_Python_ValidatesContainer_Dataflow.groovy
##########
@@ -35,11 +35,13 @@ PostcommitJobBuilder.postCommitJob('beam_PostCommit_Py_ValCont',
}
// Execute shell command to test Python SDK.
- // TODO: Parallel the script run with Jenkins DSL or Gradle.
steps {
- VALIDATES_CONTAINER_DATAFLOW_PYTHON_VERSIONS.each {
- pythonVersion = it.replace('.', '')
- shell('cd ' + commonJobProperties.checkoutDir + " && bash sdks/python/container/run_validatescontainer.sh python${pythonVersion}")
+ gradle {
+ rootBuildScriptDir(commonJobProperties.checkoutDir)
+ tasks(':sdks:python:test-suites:portable:py36:validatesContainer')
Review comment:
done.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] y1chi commented on pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
y1chi commented on pull request #14145:
URL: https://github.com/apache/beam/pull/14145#issuecomment-796416708
I think the goResolveDependencies issues should have been addressed, can we try the task again?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] codecov[bot] edited a comment on pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
codecov[bot] edited a comment on pull request #14145:
URL: https://github.com/apache/beam/pull/14145#issuecomment-790308850
# [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=h1) Report
> Merging [#14145](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=desc) (a59293d) into [master](https://codecov.io/gh/apache/beam/commit/2d9c66647ecbc9aca6a8618168d44e4c815cea19?el=desc) (2d9c666) will **increase** coverage by `0.00%`.
> The diff coverage is `n/a`.
[](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #14145 +/- ##
=======================================
Coverage 83.04% 83.05%
=======================================
Files 469 469
Lines 58472 58493 +21
=======================================
+ Hits 48556 48579 +23
+ Misses 9916 9914 -2
```
| [Impacted Files](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [.../srcs/sdks/python/apache\_beam/io/range\_trackers.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9pby9yYW5nZV90cmFja2Vycy5weQ==) | | |
| [...on/apache\_beam/ml/gcp/naturallanguageml\_test\_it.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9tbC9nY3AvbmF0dXJhbGxhbmd1YWdlbWxfdGVzdF9pdC5weQ==) | | |
| [...cs/sdks/python/apache\_beam/transforms/combiners.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS90cmFuc2Zvcm1zL2NvbWJpbmVycy5weQ==) | | |
| [...ks/python/apache\_beam/runners/worker/opcounters.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9ydW5uZXJzL3dvcmtlci9vcGNvdW50ZXJzLnB5) | | |
| [...d/srcs/sdks/python/apache\_beam/ml/gcp/cloud\_dlp.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9tbC9nY3AvY2xvdWRfZGxwLnB5) | | |
| [...cs/sdks/python/apache\_beam/io/external/\_\_init\_\_.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9pby9leHRlcm5hbC9fX2luaXRfXy5weQ==) | | |
| [...m/examples/snippets/transforms/aggregation/mean.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9leGFtcGxlcy9zbmlwcGV0cy90cmFuc2Zvcm1zL2FnZ3JlZ2F0aW9uL21lYW4ucHk=) | | |
| [...he\_beam/examples/cookbook/multiple\_output\_pardo.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9leGFtcGxlcy9jb29rYm9vay9tdWx0aXBsZV9vdXRwdXRfcGFyZG8ucHk=) | | |
| [...build/srcs/sdks/python/apache\_beam/dataframe/io.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS9kYXRhZnJhbWUvaW8ucHk=) | | |
| [.../build/srcs/sdks/python/apache\_beam/tools/utils.py](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree#diff-YmVhbV9QcmVDb21taXRfUHl0aG9uX0NvbW1pdC9zcmMvc2Rrcy9weXRob24vdGVzdC1zdWl0ZXMvdG94L3B5MzgvYnVpbGQvc3Jjcy9zZGtzL3B5dGhvbi9hcGFjaGVfYmVhbS90b29scy91dGlscy5weQ==) | | |
| ... and [928 more](https://codecov.io/gh/apache/beam/pull/14145/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=footer). Last update [1f35e2f...a59293d](https://codecov.io/gh/apache/beam/pull/14145?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [beam] tvalentyn commented on a change in pull request #14145: Parallelize python validate container tests in beam_PostCommit_Py_Val…
Posted by GitBox <gi...@apache.org>.
tvalentyn commented on a change in pull request #14145:
URL: https://github.com/apache/beam/pull/14145#discussion_r587876964
##########
File path: .test-infra/jenkins/job_PostCommit_Python_ValidatesContainer_Dataflow.groovy
##########
@@ -35,11 +35,13 @@ PostcommitJobBuilder.postCommitJob('beam_PostCommit_Py_ValCont',
}
// Execute shell command to test Python SDK.
- // TODO: Parallel the script run with Jenkins DSL or Gradle.
steps {
- VALIDATES_CONTAINER_DATAFLOW_PYTHON_VERSIONS.each {
- pythonVersion = it.replace('.', '')
- shell('cd ' + commonJobProperties.checkoutDir + " && bash sdks/python/container/run_validatescontainer.sh python${pythonVersion}")
+ gradle {
+ rootBuildScriptDir(commonJobProperties.checkoutDir)
+ tasks(':sdks:python:test-suites:portable:py36:validatesContainer')
Review comment:
I suggest we add a constant along these lines
https://github.com/apache/beam/blob/2b8938eced3b5776441246f521a38af5269f916a/sdks/python/test-suites/gradle.properties#L28
and a top-level task like so:
https://github.com/apache/beam/blob/9f9dcd6e97b29c020f555bcaa992af56108aaa0d/sdks/python/test-suites/dataflow/build.gradle#L58
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org