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/02/10 15:19:12 UTC

[GitHub] [beam] ramazan-yapparov opened a new pull request #13948: [BEAM-11589] Migrated release build.gradle file to Kotlin script

ramazan-yapparov opened a new pull request #13948:
URL: https://github.com/apache/beam/pull/13948


   [BEAM-11589] Migrated release build.gradle file to Kotlin script
   
   ------------------------
   
   Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:
   
    - [x] [**Choose reviewer(s)**](https://beam.apache.org/contribute/#make-your-change) and mention them in a comment (`R: @username`).
    - [x] 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.
    - [x] Update `CHANGES.md` with noteworthy changes.
    - [x] 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 | [![Build Status](https://ci-beam.apache.org/job/beam_PostCommit_Go/lastCompletedBuild/badge/icon)](https://ci-beam.apache.org/job/beam_PostCommit_Go/lastCompletedBuild/) | --- | [![Build Status](https://ci-beam.apache.org/job/beam_PostCommit_Go_VR_Flink/lastCompletedBuild/badge/icon)](https://ci-beam.apache.org/job/beam_PostCommit_Go_VR_Flink/lastCompletedBuild/) | --- | [![Build Status](https://ci-beam.apache.org/job/beam_PostCommit_Go_VR_Spark/lastCompletedBuild/badge/icon)](https://ci-beam.apache.org/job/beam_PostCommit_Go_VR_Spark/lastCompletedBuild/) | ---
   Java | [![Build Status](https://ci-beam.apache.org/job/beam_PostCommit_Java/lastCompletedBuild/badge/icon)](https://ci-beam.apache.org/job/beam_PostCommit_Java/lastCompletedBuild/) | [![Build Status](https://ci-beam.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Dataflow/lastCompletedBuild/badge/icon)](https://ci-beam.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Dataflow/lastCompletedBuild/)<br>[![Build Status](https://ci-beam.apache.org/job/beam_PostCommit_Java_VR_Dataflow_V2/lastCompletedBuild/badge/icon)](https://ci-beam.apache.org/job/beam_PostCommit_Java_VR_Dataflow_V2/lastCompletedBuild/)<br>[![Build Status](https://ci-beam.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Dataflow_Java11/lastCompletedBuild/badge/icon)](https://ci-beam.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Dataflow_Java11/lastCompletedBuild/) | [![Build Status](https://ci-beam.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Flink/lastCompletedBuild/badge/icon)](https://ci-beam
 .apache.org/job/beam_PostCommit_Java_ValidatesRunner_Flink/lastCompletedBuild/)<br>[![Build Status](https://ci-beam.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Flink_Java11/lastCompletedBuild/badge/icon)](https://ci-beam.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Flink_Java11/lastCompletedBuild/)<br>[![Build Status](https://ci-beam.apache.org/job/beam_PostCommit_Java_PVR_Flink_Batch/lastCompletedBuild/badge/icon)](https://ci-beam.apache.org/job/beam_PostCommit_Java_PVR_Flink_Batch/lastCompletedBuild/)<br>[![Build Status](https://ci-beam.apache.org/job/beam_PostCommit_Java_PVR_Flink_Streaming/lastCompletedBuild/badge/icon)](https://ci-beam.apache.org/job/beam_PostCommit_Java_PVR_Flink_Streaming/lastCompletedBuild/) | [![Build Status](https://ci-beam.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Samza/lastCompletedBuild/badge/icon)](https://ci-beam.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Samza/lastCompletedBuild/) | [![Build Status](https://ci-beam.a
 pache.org/job/beam_PostCommit_Java_ValidatesRunner_Spark/lastCompletedBuild/badge/icon)](https://ci-beam.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Spark/lastCompletedBuild/)<br>[![Build Status](https://ci-beam.apache.org/job/beam_PostCommit_Java_PVR_Spark_Batch/lastCompletedBuild/badge/icon)](https://ci-beam.apache.org/job/beam_PostCommit_Java_PVR_Spark_Batch/lastCompletedBuild/)<br>[![Build Status](https://ci-beam.apache.org/job/beam_PostCommit_Java_ValidatesRunner_SparkStructuredStreaming/lastCompletedBuild/badge/icon)](https://ci-beam.apache.org/job/beam_PostCommit_Java_ValidatesRunner_SparkStructuredStreaming/lastCompletedBuild/) | [![Build Status](https://ci-beam.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Twister2/lastCompletedBuild/badge/icon)](https://ci-beam.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Twister2/lastCompletedBuild/)
   Python | [![Build Status](https://ci-beam.apache.org/job/beam_PostCommit_Python36/lastCompletedBuild/badge/icon)](https://ci-beam.apache.org/job/beam_PostCommit_Python36/lastCompletedBuild/)<br>[![Build Status](https://ci-beam.apache.org/job/beam_PostCommit_Python37/lastCompletedBuild/badge/icon)](https://ci-beam.apache.org/job/beam_PostCommit_Python37/lastCompletedBuild/)<br>[![Build Status](https://ci-beam.apache.org/job/beam_PostCommit_Python38/lastCompletedBuild/badge/icon)](https://ci-beam.apache.org/job/beam_PostCommit_Python38/lastCompletedBuild/) | [![Build Status](https://ci-beam.apache.org/job/beam_PostCommit_Py_VR_Dataflow/lastCompletedBuild/badge/icon)](https://ci-beam.apache.org/job/beam_PostCommit_Py_VR_Dataflow/lastCompletedBuild/)<br>[![Build Status](https://ci-beam.apache.org/job/beam_PostCommit_Py_VR_Dataflow_V2/lastCompletedBuild/badge/icon)](https://ci-beam.apache.org/job/beam_PostCommit_Py_VR_Dataflow_V2/lastCompletedBuild/)<br>[![Build Status](https://ci-beam
 .apache.org/job/beam_PostCommit_Py_ValCont/lastCompletedBuild/badge/icon)](https://ci-beam.apache.org/job/beam_PostCommit_Py_ValCont/lastCompletedBuild/) | [![Build Status](https://ci-beam.apache.org/job/beam_PreCommit_Python_PVR_Flink_Cron/lastCompletedBuild/badge/icon)](https://ci-beam.apache.org/job/beam_PreCommit_Python_PVR_Flink_Cron/lastCompletedBuild/)<br>[![Build Status](https://ci-beam.apache.org/job/beam_PostCommit_Python_VR_Flink/lastCompletedBuild/badge/icon)](https://ci-beam.apache.org/job/beam_PostCommit_Python_VR_Flink/lastCompletedBuild/) | --- | [![Build Status](https://ci-beam.apache.org/job/beam_PostCommit_Python_VR_Spark/lastCompletedBuild/badge/icon)](https://ci-beam.apache.org/job/beam_PostCommit_Python_VR_Spark/lastCompletedBuild/) | ---
   XLang | [![Build Status](https://ci-beam.apache.org/job/beam_PostCommit_XVR_Direct/lastCompletedBuild/badge/icon)](https://ci-beam.apache.org/job/beam_PostCommit_XVR_Direct/lastCompletedBuild/) | [![Build Status](https://ci-beam.apache.org/job/beam_PostCommit_XVR_Dataflow/lastCompletedBuild/badge/icon)](https://ci-beam.apache.org/job/beam_PostCommit_XVR_Dataflow/lastCompletedBuild/) | [![Build Status](https://ci-beam.apache.org/job/beam_PostCommit_XVR_Flink/lastCompletedBuild/badge/icon)](https://ci-beam.apache.org/job/beam_PostCommit_XVR_Flink/lastCompletedBuild/) | --- | [![Build Status](https://ci-beam.apache.org/job/beam_PostCommit_XVR_Spark/lastCompletedBuild/badge/icon)](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 | [![Build Status](https://ci-beam.apache.org/job/beam_PreCommit_Java_Cron/lastCompletedBuild/badge/icon)](https://ci-beam.apache.org/job/beam_PreCommit_Java_Cron/lastCompletedBuild/) | [![Build Status](https://ci-beam.apache.org/job/beam_PreCommit_Python_Cron/lastCompletedBuild/badge/icon)](https://ci-beam.apache.org/job/beam_PreCommit_Python_Cron/lastCompletedBuild/)<br>[![Build Status](https://ci-beam.apache.org/job/beam_PreCommit_PythonLint_Cron/lastCompletedBuild/badge/icon)](https://ci-beam.apache.org/job/beam_PreCommit_PythonLint_Cron/lastCompletedBuild/)<br>[![Build Status](https://ci-beam.apache.org/job/beam_PreCommit_PythonDocker_Cron/badge/icon)](https://ci-beam.apache.org/job/beam_PreCommit_PythonDocker_Cron/lastCompletedBuild/) <br>[![Build Status](https://ci-beam.apache.org/job/beam_PreCommit_PythonDocs_Cron/badge/icon)](https://ci-beam.apache.org/job/beam_PreCommit_PythonDocs_Cron/lastCompletedBuild/) | [![Build Status](https://ci-beam.apache.org/job/be
 am_PreCommit_Go_Cron/lastCompletedBuild/badge/icon)](https://ci-beam.apache.org/job/beam_PreCommit_Go_Cron/lastCompletedBuild/) | [![Build Status](https://ci-beam.apache.org/job/beam_PreCommit_Website_Cron/lastCompletedBuild/badge/icon)](https://ci-beam.apache.org/job/beam_PreCommit_Website_Cron/lastCompletedBuild/) | [![Build Status](https://ci-beam.apache.org/job/beam_PreCommit_Whitespace_Cron/lastCompletedBuild/badge/icon)](https://ci-beam.apache.org/job/beam_PreCommit_Whitespace_Cron/lastCompletedBuild/) | [![Build Status](https://ci-beam.apache.org/job/beam_PreCommit_Typescript_Cron/lastCompletedBuild/badge/icon)](https://ci-beam.apache.org/job/beam_PreCommit_Typescript_Cron/lastCompletedBuild/)
   Portable | --- | [![Build Status](https://ci-beam.apache.org/job/beam_PreCommit_Portable_Python_Cron/lastCompletedBuild/badge/icon)](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)
   ------------------------------------------------------------------------------------------------
   [![Build python source distribution and wheels](https://github.com/apache/beam/workflows/Build%20python%20source%20distribution%20and%20wheels/badge.svg?branch=master&event=schedule)](https://github.com/apache/beam/actions?query=workflow%3A%22Build+python+source+distribution+and+wheels%22+branch%3Amaster+event%3Aschedule)
   [![Python tests](https://github.com/apache/beam/workflows/Python%20tests/badge.svg?branch=master&event=schedule)](https://github.com/apache/beam/actions?query=workflow%3A%22Python+Tests%22+branch%3Amaster+event%3Aschedule)
   [![Java tests](https://github.com/apache/beam/workflows/Java%20Tests/badge.svg?branch=master&event=schedule)](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 #13948: [BEAM-11589] Migrated release build.gradle file to Kotlin script

Posted by GitBox <gi...@apache.org>.
codecov[bot] commented on pull request #13948:
URL: https://github.com/apache/beam/pull/13948#issuecomment-776801277


   # [Codecov](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=h1) Report
   > Merging [#13948](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=desc) (022c794) into [master](https://codecov.io/gh/apache/beam/commit/827c9387d7b7efa55717d6dccde672787c72b3de?el=desc) (827c938) will **increase** coverage by `0.00%`.
   > The diff coverage is `100.00%`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/beam/pull/13948/graphs/tree.svg?width=650&height=150&src=pr&token=qcbbAh8Fj1)](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=tree)
   
   ```diff
   @@           Coverage Diff           @@
   ##           master   #13948   +/-   ##
   =======================================
     Coverage   82.85%   82.86%           
   =======================================
     Files         466      466           
     Lines       57596    57613   +17     
   =======================================
   + Hits        47723    47742   +19     
   + Misses       9873     9871    -2     
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=tree) | Coverage Δ | |
   |---|---|---|
   | [sdks/python/apache\_beam/dataframe/schemas.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vZGF0YWZyYW1lL3NjaGVtYXMucHk=) | `96.87% <100.00%> (+1.56%)` | :arrow_up: |
   | [sdks/python/apache\_beam/internal/metrics/metric.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vaW50ZXJuYWwvbWV0cmljcy9tZXRyaWMucHk=) | `86.45% <0.00%> (-1.05%)` | :arrow_down: |
   | [sdks/python/apache\_beam/runners/common.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9jb21tb24ucHk=) | `88.72% <0.00%> (-0.15%)` | :arrow_down: |
   | [sdks/python/apache\_beam/dataframe/expressions.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vZGF0YWZyYW1lL2V4cHJlc3Npb25zLnB5) | `91.02% <0.00%> (-0.14%)` | :arrow_down: |
   | [.../runners/portability/fn\_api\_runner/translations.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9wb3J0YWJpbGl0eS9mbl9hcGlfcnVubmVyL3RyYW5zbGF0aW9ucy5weQ==) | `92.10% <0.00%> (-0.08%)` | :arrow_down: |
   | [...on/apache\_beam/runners/dataflow/dataflow\_runner.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9kYXRhZmxvdy9kYXRhZmxvd19ydW5uZXIucHk=) | `82.28% <0.00%> (ø)` | |
   | [...apache\_beam/runners/portability/portable\_runner.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9wb3J0YWJpbGl0eS9wb3J0YWJsZV9ydW5uZXIucHk=) | `76.30% <0.00%> (ø)` | |
   | [...eam/runners/portability/fn\_api\_runner/fn\_runner.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9wb3J0YWJpbGl0eS9mbl9hcGlfcnVubmVyL2ZuX3J1bm5lci5weQ==) | `89.87% <0.00%> (ø)` | |
   | [sdks/python/apache\_beam/dataframe/frames.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vZGF0YWZyYW1lL2ZyYW1lcy5weQ==) | `91.07% <0.00%> (+0.13%)` | :arrow_up: |
   | [...ks/python/apache\_beam/runners/worker/sdk\_worker.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy93b3JrZXIvc2RrX3dvcmtlci5weQ==) | `89.69% <0.00%> (+0.15%)` | :arrow_up: |
   | ... and [2 more](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree-more) | |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/beam/pull/13948?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/13948?src=pr&el=footer). Last update [befcc3d...022c794](https://codecov.io/gh/apache/beam/pull/13948?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 #13948: [BEAM-11589] Migrated release build.gradle file to Kotlin script

Posted by GitBox <gi...@apache.org>.
codecov[bot] edited a comment on pull request #13948:
URL: https://github.com/apache/beam/pull/13948#issuecomment-776801277


   # [Codecov](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=h1) Report
   > Merging [#13948](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=desc) (022c794) into [master](https://codecov.io/gh/apache/beam/commit/827c9387d7b7efa55717d6dccde672787c72b3de?el=desc) (827c938) will **increase** coverage by `0.00%`.
   > The diff coverage is `100.00%`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/beam/pull/13948/graphs/tree.svg?width=650&height=150&src=pr&token=qcbbAh8Fj1)](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=tree)
   
   ```diff
   @@           Coverage Diff           @@
   ##           master   #13948   +/-   ##
   =======================================
     Coverage   82.85%   82.86%           
   =======================================
     Files         466      466           
     Lines       57596    57613   +17     
   =======================================
   + Hits        47723    47742   +19     
   + Misses       9873     9871    -2     
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=tree) | Coverage Δ | |
   |---|---|---|
   | [sdks/python/apache\_beam/dataframe/schemas.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vZGF0YWZyYW1lL3NjaGVtYXMucHk=) | `96.87% <100.00%> (+1.56%)` | :arrow_up: |
   | [sdks/python/apache\_beam/internal/metrics/metric.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vaW50ZXJuYWwvbWV0cmljcy9tZXRyaWMucHk=) | `86.45% <0.00%> (-1.05%)` | :arrow_down: |
   | [sdks/python/apache\_beam/runners/common.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9jb21tb24ucHk=) | `88.72% <0.00%> (-0.15%)` | :arrow_down: |
   | [sdks/python/apache\_beam/dataframe/expressions.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vZGF0YWZyYW1lL2V4cHJlc3Npb25zLnB5) | `91.02% <0.00%> (-0.14%)` | :arrow_down: |
   | [.../runners/portability/fn\_api\_runner/translations.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9wb3J0YWJpbGl0eS9mbl9hcGlfcnVubmVyL3RyYW5zbGF0aW9ucy5weQ==) | `92.10% <0.00%> (-0.08%)` | :arrow_down: |
   | [...on/apache\_beam/runners/dataflow/dataflow\_runner.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9kYXRhZmxvdy9kYXRhZmxvd19ydW5uZXIucHk=) | `82.28% <0.00%> (ø)` | |
   | [...apache\_beam/runners/portability/portable\_runner.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9wb3J0YWJpbGl0eS9wb3J0YWJsZV9ydW5uZXIucHk=) | `76.30% <0.00%> (ø)` | |
   | [...eam/runners/portability/fn\_api\_runner/fn\_runner.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9wb3J0YWJpbGl0eS9mbl9hcGlfcnVubmVyL2ZuX3J1bm5lci5weQ==) | `89.87% <0.00%> (ø)` | |
   | [sdks/python/apache\_beam/dataframe/frames.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vZGF0YWZyYW1lL2ZyYW1lcy5weQ==) | `91.07% <0.00%> (+0.13%)` | :arrow_up: |
   | [...ks/python/apache\_beam/runners/worker/sdk\_worker.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy93b3JrZXIvc2RrX3dvcmtlci5weQ==) | `89.69% <0.00%> (+0.15%)` | :arrow_up: |
   | ... and [2 more](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree-more) | |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/beam/pull/13948?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/13948?src=pr&el=footer). Last update [befcc3d...022c794](https://codecov.io/gh/apache/beam/pull/13948?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 #13948: [BEAM-11589] Migrated release build.gradle file to Kotlin script

Posted by GitBox <gi...@apache.org>.
codecov[bot] edited a comment on pull request #13948:
URL: https://github.com/apache/beam/pull/13948#issuecomment-776801277


   # [Codecov](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=h1) Report
   > Merging [#13948](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=desc) (c8f3b51) into [master](https://codecov.io/gh/apache/beam/commit/9aa9c8625ad3a1908dd300c4b7b949fc51828466?el=desc) (9aa9c86) will **not change** coverage.
   > The diff coverage is `n/a`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/beam/pull/13948/graphs/tree.svg?width=650&height=150&src=pr&token=qcbbAh8Fj1)](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=tree)
   
   ```diff
   @@           Coverage Diff           @@
   ##           master   #13948   +/-   ##
   =======================================
     Coverage   82.94%   82.94%           
   =======================================
     Files         469      469           
     Lines       58309    58309           
   =======================================
     Hits        48367    48367           
     Misses       9942     9942           
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=tree) | Coverage Δ | |
   |---|---|---|
   | [sdks/python/apache\_beam/internal/metrics/metric.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vaW50ZXJuYWwvbWV0cmljcy9tZXRyaWMucHk=) | `86.45% <0.00%> (-1.05%)` | :arrow_down: |
   | [...nners/portability/fn\_api\_runner/worker\_handlers.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9wb3J0YWJpbGl0eS9mbl9hcGlfcnVubmVyL3dvcmtlcl9oYW5kbGVycy5weQ==) | `80.00% <0.00%> (-0.36%)` | :arrow_down: |
   | [...hon/apache\_beam/runners/worker/bundle\_processor.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy93b3JrZXIvYnVuZGxlX3Byb2Nlc3Nvci5weQ==) | `93.86% <0.00%> (-0.13%)` | :arrow_down: |
   | [.../python/apache\_beam/transforms/periodicsequence.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vdHJhbnNmb3Jtcy9wZXJpb2RpY3NlcXVlbmNlLnB5) | `98.24% <0.00%> (+1.75%)` | :arrow_up: |
   | [...hon/apache\_beam/runners/direct/test\_stream\_impl.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9kaXJlY3QvdGVzdF9zdHJlYW1faW1wbC5weQ==) | `94.11% <0.00%> (+2.20%)` | :arrow_up: |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/beam/pull/13948?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/13948?src=pr&el=footer). Last update [9aa9c86...c8f3b51](https://codecov.io/gh/apache/beam/pull/13948?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 #13948: [BEAM-11589] Migrated release build.gradle file to Kotlin script

Posted by GitBox <gi...@apache.org>.
codecov[bot] edited a comment on pull request #13948:
URL: https://github.com/apache/beam/pull/13948#issuecomment-776801277


   # [Codecov](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=h1) Report
   > Merging [#13948](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=desc) (022c794) into [master](https://codecov.io/gh/apache/beam/commit/9aa9c8625ad3a1908dd300c4b7b949fc51828466?el=desc) (9aa9c86) will **decrease** coverage by `0.07%`.
   > The diff coverage is `n/a`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/beam/pull/13948/graphs/tree.svg?width=650&height=150&src=pr&token=qcbbAh8Fj1)](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=tree)
   
   ```diff
   @@            Coverage Diff             @@
   ##           master   #13948      +/-   ##
   ==========================================
   - Coverage   82.94%   82.87%   -0.08%     
   ==========================================
     Files         469      466       -3     
     Lines       58309    57613     -696     
   ==========================================
   - Hits        48367    47744     -623     
   + Misses       9942     9869      -73     
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=tree) | Coverage Δ | |
   |---|---|---|
   | [sdks/python/apache\_beam/coders/slow\_stream.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vY29kZXJzL3Nsb3dfc3RyZWFtLnB5) | `92.43% <0.00%> (-2.21%)` | :arrow_down: |
   | [sdks/python/apache\_beam/dataframe/frame\_base.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vZGF0YWZyYW1lL2ZyYW1lX2Jhc2UucHk=) | `88.46% <0.00%> (-2.09%)` | :arrow_down: |
   | [...eam/runners/portability/fn\_api\_runner/execution.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9wb3J0YWJpbGl0eS9mbl9hcGlfcnVubmVyL2V4ZWN1dGlvbi5weQ==) | `92.03% <0.00%> (-1.65%)` | :arrow_down: |
   | [sdks/python/apache\_beam/internal/metrics/metric.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vaW50ZXJuYWwvbWV0cmljcy9tZXRyaWMucHk=) | `86.45% <0.00%> (-1.05%)` | :arrow_down: |
   | [...dks/python/apache\_beam/runners/pipeline\_context.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9waXBlbGluZV9jb250ZXh0LnB5) | `92.74% <0.00%> (-0.81%)` | :arrow_down: |
   | [sdks/python/apache\_beam/internal/pickler.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vaW50ZXJuYWwvcGlja2xlci5weQ==) | `86.33% <0.00%> (-0.54%)` | :arrow_down: |
   | [...ks/python/apache\_beam/coders/coders\_test\_common.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vY29kZXJzL2NvZGVyc190ZXN0X2NvbW1vbi5weQ==) | `98.52% <0.00%> (-0.36%)` | :arrow_down: |
   | [...nners/portability/fn\_api\_runner/worker\_handlers.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9wb3J0YWJpbGl0eS9mbl9hcGlfcnVubmVyL3dvcmtlcl9oYW5kbGVycy5weQ==) | `80.00% <0.00%> (-0.36%)` | :arrow_down: |
   | [sdks/python/apache\_beam/coders/coders.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vY29kZXJzL2NvZGVycy5weQ==) | `87.26% <0.00%> (-0.22%)` | :arrow_down: |
   | [sdks/python/apache\_beam/coders/coder\_impl.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vY29kZXJzL2NvZGVyX2ltcGwucHk=) | `95.38% <0.00%> (-0.14%)` | :arrow_down: |
   | ... and [11 more](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree-more) | |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/beam/pull/13948?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/13948?src=pr&el=footer). Last update [9aa9c86...3926c3c](https://codecov.io/gh/apache/beam/pull/13948?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] TheNeuralBit commented on a change in pull request #13948: [BEAM-11589] Migrated release build.gradle file to Kotlin script

Posted by GitBox <gi...@apache.org>.
TheNeuralBit commented on a change in pull request #13948:
URL: https://github.com/apache/beam/pull/13948#discussion_r575506725



##########
File path: buildSrc/build.gradle.kts
##########
@@ -21,6 +21,7 @@ plugins {
   `java-gradle-plugin`
   groovy
   id("com.diffplug.spotless") version "5.6.1"
+  `kotlin-dsl`

Review comment:
       Is this a mistake?

##########
File path: buildSrc/src/main/kotlin/org/apache/beam/gradle/BeamDependencies.kt
##########
@@ -0,0 +1,253 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * License); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an AS IS BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package org.apache.beam.gradle
+
+/** ***********************************************************************************************/
+// Define and export a map dependencies shared across multiple sub-projects.
+//
+// Example usage:
+// configuration {
+//   compile(library.java.avro)
+//   testCompile(library.java.junit)
+// }
+
+// These versions are defined here because they represent
+// a dependency version which should match across multiple
+// Maven artifacts.
+const val activemq_version = "5.14.5"
+const val autovalue_version = "1.7.2"
+const val aws_java_sdk_version = "1.11.718"
+const val aws_java_sdk2_version = "2.13.54"
+const val cassandra_driver_version = "3.10.2"
+const val checkerframework_version = "3.7.0"
+const val classgraph_version = "4.8.65"
+const val google_clients_version = "1.30.10"
+const val google_cloud_bigdataoss_version = "2.1.6"
+const val google_cloud_pubsub_version = "1.108.6"
+const val google_cloud_pubsublite_version = "0.7.0"
+const val google_code_gson_version = "2.8.6"
+const val google_oauth_clients_version = "1.31.0"
+// Try to keep grpc_version consistent with gRPC version in google_cloud_platform_libraries_bom
+const val grpc_version = "1.32.2"
+const val guava_version = "25.1-jre"
+const val hadoop_version = "2.10.1"
+const val hamcrest_version = "2.1"
+const val influxdb_version = "2.19"
+const val httpclient_version = "4.5.10"
+const val httpcore_version = "4.4.12"
+const val jackson_version = "2.10.2"
+const val jaxb_api_version = "2.3.3"
+const val jsr305_version = "3.0.2"
+const val kafka_version = "2.4.1"
+const val nemo_version = "0.1"
+const val netty_version = "4.1.51.Final"
+const val postgres_version = "42.2.16"
+const val powermock_version = "2.0.2"
+const val protobuf_version = "3.12.0"
+const val quickcheck_version = "0.8"
+const val slf4j_version = "1.7.30"
+const val spark_version = "2.4.7"
+const val spotbugs_version = "4.0.6"
+const val testcontainers_version = "1.15.1"
+
+// A map of maps containing common libraries used per language. To use:
+// dependencies {
+//   compile(library.java.slf4j_api)
+// }
+object library {
+    object java {
+        const val activemq_amqp                               = "org.apache.activemq:activemq-amqp:$activemq_version"
+        const val activemq_broker                             = "org.apache.activemq:activemq-broker:$activemq_version"
+        const val activemq_client                             = "org.apache.activemq:activemq-client:$activemq_version"
+        const val activemq_jaas                               = "org.apache.activemq:activemq-jaas:$activemq_version"
+        const val activemq_junit                              = "org.apache.activemq.tooling:activemq-junit:$activemq_version"
+        const val activemq_kahadb_store                       = "org.apache.activemq:activemq-kahadb-store:$activemq_version"
+        const val activemq_mqtt                               = "org.apache.activemq:activemq-mqtt:$activemq_version"
+        const val antlr                                       = "org.antlr:antlr4:4.7"
+        const val antlr_runtime                               = "org.antlr:antlr4-runtime:4.7"
+        const val args4j                                      = "args4j:args4j:2.33"
+        const val avro                                        = "org.apache.avro:avro:1.8.2"
+        const val avro_tests                                  = "org.apache.avro:avro:1.8.2:tests"
+        const val aws_java_sdk_cloudwatch                     = "com.amazonaws:aws-java-sdk-cloudwatch:$aws_java_sdk_version"
+        const val aws_java_sdk_core                           = "com.amazonaws:aws-java-sdk-core:$aws_java_sdk_version"
+        const val aws_java_sdk_dynamodb                       = "com.amazonaws:aws-java-sdk-dynamodb:$aws_java_sdk_version"
+        const val aws_java_sdk_kinesis                        = "com.amazonaws:aws-java-sdk-kinesis:$aws_java_sdk_version"
+        const val aws_java_sdk_s3                             = "com.amazonaws:aws-java-sdk-s3:$aws_java_sdk_version"
+        const val aws_java_sdk_sns                            = "com.amazonaws:aws-java-sdk-sns:$aws_java_sdk_version"
+        const val aws_java_sdk_sqs                            = "com.amazonaws:aws-java-sdk-sqs:$aws_java_sdk_version"
+        const val aws_java_sdk_sts                            = "com.amazonaws:aws-java-sdk-sts:$aws_java_sdk_version"
+        const val aws_java_sdk2_apache_client                 = "software.amazon.awssdk:apache-client:$aws_java_sdk2_version"
+        const val aws_java_sdk2_auth                          = "software.amazon.awssdk:auth:$aws_java_sdk2_version"
+        const val aws_java_sdk2_cloudwatch                    = "software.amazon.awssdk:cloudwatch:$aws_java_sdk2_version"
+        const val aws_java_sdk2_dynamodb                      = "software.amazon.awssdk:dynamodb:$aws_java_sdk2_version"
+        const val aws_java_sdk2_kinesis                       = "software.amazon.awssdk:kinesis:$aws_java_sdk2_version"
+        const val aws_java_sdk2_sdk_core                      = "software.amazon.awssdk:sdk-core:$aws_java_sdk2_version"
+        const val aws_java_sdk2_sns                           = "software.amazon.awssdk:sns:$aws_java_sdk2_version"
+        const val aws_java_sdk2_sqs                           = "software.amazon.awssdk:sqs:$aws_java_sdk2_version"
+        const val bigdataoss_gcsio                            = "com.google.cloud.bigdataoss:gcsio:$google_cloud_bigdataoss_version"
+        const val bigdataoss_util                             = "com.google.cloud.bigdataoss:util:$google_cloud_bigdataoss_version"
+        const val cassandra_driver_core                       = "com.datastax.cassandra:cassandra-driver-core:$cassandra_driver_version"
+        const val cassandra_driver_mapping                    = "com.datastax.cassandra:cassandra-driver-mapping:$cassandra_driver_version"
+        const val classgraph                                  = "io.github.classgraph:classgraph:$classgraph_version"
+        const val commons_codec                               = "commons-codec:commons-codec:1.14"
+        const val commons_compress                            = "org.apache.commons:commons-compress:1.20"
+        const val commons_csv                                 = "org.apache.commons:commons-csv:1.8"
+        const val commons_io                                  = "commons-io:commons-io:2.6"
+        const val commons_lang3                               = "org.apache.commons:commons-lang3:3.9"
+        const val commons_math3                               = "org.apache.commons:commons-math3:3.6.1"
+        const val error_prone_annotations                     = "com.google.errorprone:error_prone_annotations:2.3.1"
+        const val gax                                         = "com.google.api:gax" // google_cloud_platform_libraries_bom sets versio
+        const val gax_grpc                                    = "com.google.api:gax-grpc" // google_cloud_platform_libraries_bom sets versio
+        const val google_api_client                           = "com.google.api-client:google-api-client:$google_clients_version"
+        const val google_api_client_jackson2                  = "com.google.api-client:google-api-client-jackson2:$google_clients_version"
+        const val google_api_client_java6                     = "com.google.api-client:google-api-client-java6:$google_clients_version"
+        const val google_api_common                           = "com.google.api:api-common" // google_cloud_platform_libraries_bom sets versio
+        const val google_api_services_bigquery                = "com.google.apis:google-api-services-bigquery:v2-rev20200719-$google_clients_version"
+        const val google_api_services_clouddebugger           = "com.google.apis:google-api-services-clouddebugger:v2-rev20200501-$google_clients_version"
+        const val google_api_services_cloudresourcemanager    = "com.google.apis:google-api-services-cloudresourcemanager:v1-rev20200720-$google_clients_version"
+        const val google_api_services_dataflow                = "com.google.apis:google-api-services-dataflow:v1b3-rev20200713-$google_clients_version"
+        const val google_api_services_healthcare              = "com.google.apis:google-api-services-healthcare:v1beta1-rev20200713-$google_clients_version"
+        const val google_api_services_pubsub                  = "com.google.apis:google-api-services-pubsub:v1-rev20200713-$google_clients_version"
+        const val google_api_services_storage                 = "com.google.apis:google-api-services-storage:v1-rev20200611-$google_clients_version"
+        const val google_auth_library_credentials             = "com.google.auth:google-auth-library-credentials" // google_cloud_platform_libraries_bom sets versio
+        const val google_auth_library_oauth2_http             = "com.google.auth:google-auth-library-oauth2-http" // google_cloud_platform_libraries_bom sets versio
+        const val google_cloud_bigquery                       = "com.google.cloud:google-cloud-bigquery" // google_cloud_platform_libraries_bom sets versio
+        const val google_cloud_bigquery_storage               = "com.google.cloud:google-cloud-bigquerystorage" // google_cloud_platform_libraries_bom sets versio
+        const val google_cloud_bigtable_client_core           = "com.google.cloud.bigtable:bigtable-client-core:1.16.0"
+        const val google_cloud_bigtable_emulator              = "com.google.cloud:google-cloud-bigtable-emulator:0.125.2"
+        const val google_cloud_core                           = "com.google.cloud:google-cloud-core" // google_cloud_platform_libraries_bom sets versio
+        const val google_cloud_core_grpc                      = "com.google.cloud:google-cloud-core-grpc" // google_cloud_platform_libraries_bom sets versio
+        const val google_cloud_datacatalog_v1beta1            = "com.google.cloud:google-cloud-datacatalog" // google_cloud_platform_libraries_bom sets versio
+        const val google_cloud_dataflow_java_proto_library_all= "com.google.cloud.dataflow:google-cloud-dataflow-java-proto-library-all:0.5.160304"
+        const val google_cloud_datastore_v1_proto_client      = "com.google.cloud.datastore:datastore-v1-proto-client:1.6.3"
+        const val google_cloud_pubsub                         = "com.google.cloud:google-cloud-pubsub:$google_cloud_pubsub_version"
+        const val google_cloud_pubsublite                     = "com.google.cloud:google-cloud-pubsublite:$google_cloud_pubsublite_version"
+// The GCP Libraries BOM dashboard shows the versions set by the BOM:
+// https://storage.googleapis.com/cloud-opensource-java-dashboard/com.google.cloud/libraries-bom/13.2.0/artifact_details.html
+// Update libraries-bom version on sdks/java/container/license_scripts/dep_urls_java.yaml
+        const val google_cloud_platform_libraries_bom         = "com.google.cloud:libraries-bom:13.2.0"
+        const val google_cloud_spanner                        = "com.google.cloud:google-cloud-spanner" // google_cloud_platform_libraries_bom sets versio
+        const val google_code_gson                            = "com.google.code.gson:gson:$google_code_gson_version"
+// google-http-client's version is explicitly declared for sdks/java/maven-archetypes/examples
+// This version should be in line with the one in com.google.cloud:libraries-bom.
+        const val google_http_client                          = "com.google.http-client:google-http-client" // google_cloud_platform_libraries_bom sets versio
+        const val google_http_client_apache_v2                = "com.google.http-client:google-http-client-apache-v2" // google_cloud_platform_libraries_bom sets versio
+        const val google_http_client_jackson                  = "com.google.http-client:google-http-client-jackson:1.29.2"
+        const val google_http_client_jackson2                 = "com.google.http-client:google-http-client-jackson2" // google_cloud_platform_libraries_bom sets versio
+        const val google_http_client_protobuf                 = "com.google.http-client:google-http-client-protobuf" // google_cloud_platform_libraries_bom sets versio
+        const val google_oauth_client                         = "com.google.oauth-client:google-oauth-client:$google_oauth_clients_version"
+        const val google_oauth_client_java6                   = "com.google.oauth-client:google-oauth-client-java6:$google_oauth_clients_version"
+// Don'tconst val  use grpc_all, it can cause issues in Bazel builds. Reference the gRPC libraries you need individually instead.
+        const val grpc_alts                                   = "io.grpc:grpc-alts" // google_cloud_platform_libraries_bom sets versio
+        const val grpc_api                                    = "io.grpc:grpc-api" // google_cloud_platform_libraries_bom sets versio
+        const val grpc_auth                                   = "io.grpc:grpc-auth" // google_cloud_platform_libraries_bom sets versio
+        const val grpc_context                                = "io.grpc:grpc-context" // google_cloud_platform_libraries_bom sets versio
+        const val grpc_core                                   = "io.grpc:grpc-core" // google_cloud_platform_libraries_bom sets versio
+        const val grpc_google_cloud_pubsub_v1                 = "com.google.api.grpc:grpc-google-cloud-pubsub-v1" // google_cloud_platform_libraries_bom sets versio
+        const val grpc_google_cloud_pubsublite_v1             = "com.google.api.grpc:grpc-google-cloud-pubsublite-v1:$google_cloud_pubsublite_version"
+        const val grpc_grpclb                                 = "io.grpc:grpc-grpclb" // google_cloud_platform_libraries_bom sets versio
+        const val grpc_protobuf                               = "io.grpc:grpc-protobuf" // google_cloud_platform_libraries_bom sets versio
+        const val grpc_protobuf_lite                          = "io.grpc:grpc-protobuf-lite:$grpc_version"
+        const val grpc_netty                                  = "io.grpc:grpc-netty" // google_cloud_platform_libraries_bom sets versio
+        const val grpc_netty_shaded                           = "io.grpc:grpc-netty-shaded" // google_cloud_platform_libraries_bom sets versio
+        const val grpc_stub                                   = "io.grpc:grpc-stub" // google_cloud_platform_libraries_bom sets versio
+        const val guava                                       = "com.google.guava:guava:$guava_version"
+        const val guava_testlib                               = "com.google.guava:guava-testlib:$guava_version"
+        const val hadoop_client                               = "org.apache.hadoop:hadoop-client:$hadoop_version"
+        const val hadoop_common                               = "org.apache.hadoop:hadoop-common:$hadoop_version"
+        const val hadoop_mapreduce_client_core                = "org.apache.hadoop:hadoop-mapreduce-client-core:$hadoop_version"
+        const val hadoop_minicluster                          = "org.apache.hadoop:hadoop-minicluster:$hadoop_version"
+        const val hadoop_hdfs                                 = "org.apache.hadoop:hadoop-hdfs:$hadoop_version"
+        const val hadoop_hdfs_tests                           = "org.apache.hadoop:hadoop-hdfs:$hadoop_version:tests"
+        const val hamcrest_core                               = "org.hamcrest:hamcrest-core:$hamcrest_version"
+        const val hamcrest_library                            = "org.hamcrest:hamcrest-library:$hamcrest_version"
+        const val http_client                                 = "org.apache.httpcomponents:httpclient:$httpclient_version"
+        const val http_core                                   = "org.apache.httpcomponents:httpcore:$httpcore_version"
+        const val influxdb_library                            = "org.influxdb:influxdb-java:$influxdb_version"
+        const val jackson_annotations                         = "com.fasterxml.jackson.core:jackson-annotations:$jackson_version"
+        const val jackson_jaxb_annotations                    = "com.fasterxml.jackson.module:jackson-module-jaxb-annotations:$jackson_version"
+        const val jackson_core                                = "com.fasterxml.jackson.core:jackson-core:$jackson_version"
+        const val jackson_databind                            = "com.fasterxml.jackson.core:jackson-databind:$jackson_version"
+        const val jackson_dataformat_cbor                     = "com.fasterxml.jackson.dataformat:jackson-dataformat-cbor:$jackson_version"
+        const val jackson_dataformat_csv                      = "com.fasterxml.jackson.dataformat:jackson-dataformat-csv:$jackson_version"
+        const val jackson_dataformat_xml                      = "com.fasterxml.jackson.dataformat:jackson-dataformat-xml:$jackson_version"
+        const val jackson_dataformat_yaml                     = "com.fasterxml.jackson.dataformat:jackson-dataformat-yaml:$jackson_version"
+        const val jackson_datatype_joda                       = "com.fasterxml.jackson.datatype:jackson-datatype-joda:$jackson_version"
+        const val jackson_module_scala                        = "com.fasterxml.jackson.module:jackson-module-scala_2.11:$jackson_version"
+        const val jaxb_api                                    = "jakarta.xml.bind:jakarta.xml.bind-api:$jaxb_api_version"
+        const val jaxb_impl                                   = "com.sun.xml.bind:jaxb-impl:$jaxb_api_version"
+        const val joda_time                                   = "joda-time:joda-time:2.10.5"
+        const val jsonassert                                  = "org.skyscreamer:jsonassert:1.5.0"
+        const val jsr305                                      = "com.google.code.findbugs:jsr305:$jsr305_version"
+        const val junit                                       = "junit:junit:4.13.1"
+        const val kafka                                       = "org.apache.kafka:kafka_2.11:$kafka_version"
+        const val kafka_clients                               = "org.apache.kafka:kafka-clients:$kafka_version"
+        const val mockito_core                                = "org.mockito:mockito-core:3.7.7"
+        const val nemo_compiler_frontend_beam                 = "org.apache.nemo:nemo-compiler-frontend-beam:$nemo_version"
+        const val netty_all                                   = "io.netty:netty-all:$netty_version"
+        const val netty_handler                               = "io.netty:netty-handler:$netty_version"
+        const val netty_tcnative_boringssl_static             = "io.netty:netty-tcnative-boringssl-static:2.0.33.Final"
+        const val netty_transport_native_epoll                = "io.netty:netty-transport-native-epoll:$netty_version"
+        const val postgres                                    = "org.postgresql:postgresql:$postgres_version"
+        const val powermock                                   = "org.powermock:powermock-module-junit4:$powermock_version"
+        const val powermock_mockito                           = "org.powermock:powermock-api-mockito2:$powermock_version"
+        const val protobuf_java                               = "com.google.protobuf:protobuf-java:$protobuf_version"
+        const val protobuf_java_util                          = "com.google.protobuf:protobuf-java-util:$protobuf_version"
+        const val proto_google_cloud_bigquery_storage_v1beta1 = "com.google.api.grpc:proto-google-cloud-bigquerystorage-v1beta1" // google_cloud_platform_libraries_bom sets versio
+        const val proto_google_cloud_bigtable_v2              = "com.google.api.grpc:proto-google-cloud-bigtable-v2" // google_cloud_platform_libraries_bom sets versio
+        const val proto_google_cloud_datastore_v1             = "com.google.api.grpc:proto-google-cloud-datastore-v1" // google_cloud_platform_libraries_bom sets versio
+        const val proto_google_cloud_pubsub_v1                = "com.google.api.grpc:proto-google-cloud-pubsub-v1" // google_cloud_platform_libraries_bom sets versio
+        const val proto_google_cloud_pubsublite_v1            = "com.google.api.grpc:proto-google-cloud-pubsublite-v1:$google_cloud_pubsublite_version"
+        const val proto_google_cloud_spanner_admin_database_v1= "com.google.api.grpc:proto-google-cloud-spanner-admin-database-v1" // google_cloud_platform_libraries_bom sets versio
+        const val proto_google_common_protos                  = "com.google.api.grpc:proto-google-common-protos" // google_cloud_platform_libraries_bom sets versio
+        const val slf4j_api                                   = "org.slf4j:slf4j-api:$slf4j_version"
+        const val slf4j_simple                                = "org.slf4j:slf4j-simple:$slf4j_version"
+        const val slf4j_jdk14                                 = "org.slf4j:slf4j-jdk14:$slf4j_version"
+        const val slf4j_log4j12                               = "org.slf4j:slf4j-log4j12:$slf4j_version"
+        const val snappy_java                                 = "org.xerial.snappy:snappy-java:1.1.4"
+        const val spark_core                                  = "org.apache.spark:spark-core_2.11:$spark_version"
+        const val spark_network_common                        = "org.apache.spark:spark-network-common_2.11:$spark_version"
+        const val spark_sql                                   = "org.apache.spark:spark-sql_2.11:$spark_version"
+        const val spark_streaming                             = "org.apache.spark:spark-streaming_2.11:$spark_version"
+        const val stax2_api                                   = "org.codehaus.woodstox:stax2-api:4.2.1"
+        const val testcontainers_clickhouse                   = "org.testcontainers:clickhouse:$testcontainers_version"
+        const val testcontainers_elasticsearch                = "org.testcontainers:elasticsearch:$testcontainers_version"
+        const val testcontainers_kafka                        = "org.testcontainers:kafka:$testcontainers_version"
+        const val testcontainers_localstack                   = "org.testcontainers:localstack:$testcontainers_version"
+        const val testcontainers_postgresql                   = "org.testcontainers:postgresql:$testcontainers_version"
+        const val testcontainers_gcloud                       = "org.testcontainers:gcloud:$testcontainers_version"
+        const val vendored_bytebuddy_1_10_8                   = "org.apache.beam:beam-vendor-bytebuddy-1_10_8:0.1"
+        const val vendored_grpc_1_26_0                        = "org.apache.beam:beam-vendor-grpc-1_26_0:0.3"
+        const val vendored_guava_26_0_jre                     = "org.apache.beam:beam-vendor-guava-26_0-jre:0.1"
+        const val vendored_calcite_1_20_0                     = "org.apache.beam:beam-vendor-calcite-1_20_0:0.1"
+        const val woodstox_core_asl                           = "org.codehaus.woodstox:woodstox-core-asl:4.4.1"
+        const val zstd_jni                                    = "com.github.luben:zstd-jni:1.4.5-2"
+        const val quickcheck_core                             = "com.pholser:junit-quickcheck-core:$quickcheck_version"
+    }
+    object groovy {
+        const val groovy_all = "org.codehaus.groovy:groovy-all:2.4.13"
+    }
+    object maven {
+        const val maven_compiler_plugin = "maven-plugins:maven-compiler-plugin:3.7.0"
+        const val maven_exec_plugin     = "maven-plugins:maven-exec-plugin:1.6.0"
+        const val maven_jar_plugin      = "maven-plugins:maven-jar-plugin:3.0.2"
+        const val maven_shade_plugin    = "maven-plugins:maven-shade-plugin:3.1.0"
+        const val maven_surefire_plugin = "maven-plugins:maven-surefire-plugin:3.0.0-M5"
+    }

Review comment:
       Is it possible to also remove `libraries` from `BeamModulePlugin.groovy`?
   
   If the only way to do this is to duplicate the definitions, I don't think we should. These dependencies change quite often so the two definitions would start diverging.




----------------------------------------------------------------
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] ramazan-yapparov commented on pull request #13948: [BEAM-11589] Migrated release build.gradle file to Kotlin script

Posted by GitBox <gi...@apache.org>.
ramazan-yapparov commented on pull request #13948:
URL: https://github.com/apache/beam/pull/13948#issuecomment-777063746


   Run Python PreCommit


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [beam] codecov[bot] edited a comment on pull request #13948: [BEAM-11589] Migrated release build.gradle file to Kotlin script

Posted by GitBox <gi...@apache.org>.
codecov[bot] edited a comment on pull request #13948:
URL: https://github.com/apache/beam/pull/13948#issuecomment-776801277


   # [Codecov](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=h1) Report
   > Merging [#13948](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=desc) (022c794) into [master](https://codecov.io/gh/apache/beam/commit/9aa9c8625ad3a1908dd300c4b7b949fc51828466?el=desc) (9aa9c86) will **decrease** coverage by `0.07%`.
   > The diff coverage is `n/a`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/beam/pull/13948/graphs/tree.svg?width=650&height=150&src=pr&token=qcbbAh8Fj1)](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=tree)
   
   ```diff
   @@            Coverage Diff             @@
   ##           master   #13948      +/-   ##
   ==========================================
   - Coverage   82.94%   82.87%   -0.08%     
   ==========================================
     Files         469      466       -3     
     Lines       58309    57613     -696     
   ==========================================
   - Hits        48367    47744     -623     
   + Misses       9942     9869      -73     
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=tree) | Coverage Δ | |
   |---|---|---|
   | [sdks/python/apache\_beam/coders/slow\_stream.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vY29kZXJzL3Nsb3dfc3RyZWFtLnB5) | `92.43% <0.00%> (-2.21%)` | :arrow_down: |
   | [sdks/python/apache\_beam/dataframe/frame\_base.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vZGF0YWZyYW1lL2ZyYW1lX2Jhc2UucHk=) | `88.46% <0.00%> (-2.09%)` | :arrow_down: |
   | [...eam/runners/portability/fn\_api\_runner/execution.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9wb3J0YWJpbGl0eS9mbl9hcGlfcnVubmVyL2V4ZWN1dGlvbi5weQ==) | `92.03% <0.00%> (-1.65%)` | :arrow_down: |
   | [sdks/python/apache\_beam/internal/metrics/metric.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vaW50ZXJuYWwvbWV0cmljcy9tZXRyaWMucHk=) | `86.45% <0.00%> (-1.05%)` | :arrow_down: |
   | [...dks/python/apache\_beam/runners/pipeline\_context.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9waXBlbGluZV9jb250ZXh0LnB5) | `92.74% <0.00%> (-0.81%)` | :arrow_down: |
   | [sdks/python/apache\_beam/internal/pickler.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vaW50ZXJuYWwvcGlja2xlci5weQ==) | `86.33% <0.00%> (-0.54%)` | :arrow_down: |
   | [...ks/python/apache\_beam/coders/coders\_test\_common.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vY29kZXJzL2NvZGVyc190ZXN0X2NvbW1vbi5weQ==) | `98.52% <0.00%> (-0.36%)` | :arrow_down: |
   | [...nners/portability/fn\_api\_runner/worker\_handlers.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9wb3J0YWJpbGl0eS9mbl9hcGlfcnVubmVyL3dvcmtlcl9oYW5kbGVycy5weQ==) | `80.00% <0.00%> (-0.36%)` | :arrow_down: |
   | [sdks/python/apache\_beam/coders/coders.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vY29kZXJzL2NvZGVycy5weQ==) | `87.26% <0.00%> (-0.22%)` | :arrow_down: |
   | [sdks/python/apache\_beam/coders/coder\_impl.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vY29kZXJzL2NvZGVyX2ltcGwucHk=) | `95.38% <0.00%> (-0.14%)` | :arrow_down: |
   | ... and [11 more](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree-more) | |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/beam/pull/13948?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/13948?src=pr&el=footer). Last update [9aa9c86...ed8e86c](https://codecov.io/gh/apache/beam/pull/13948?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] ramazan-yapparov commented on pull request #13948: [BEAM-11589] Migrated release build.gradle file to Kotlin script

Posted by GitBox <gi...@apache.org>.
ramazan-yapparov commented on pull request #13948:
URL: https://github.com/apache/beam/pull/13948#issuecomment-779467116


   Run Java PreCommit


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [beam] TheNeuralBit merged pull request #13948: [BEAM-11589] Migrated release build.gradle file to Kotlin script

Posted by GitBox <gi...@apache.org>.
TheNeuralBit merged pull request #13948:
URL: https://github.com/apache/beam/pull/13948


   


----------------------------------------------------------------
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] TheNeuralBit commented on pull request #13948: [BEAM-11589] Migrated release build.gradle file to Kotlin script

Posted by GitBox <gi...@apache.org>.
TheNeuralBit commented on pull request #13948:
URL: https://github.com/apache/beam/pull/13948#issuecomment-777669466


   > No, I can do it now - move only libraries and leave the rest as is
   
   Oh I see. Sure let's go ahead and do that, assuming you can do it without changing all the existing `libraries` usages.


----------------------------------------------------------------
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] TheNeuralBit commented on a change in pull request #13948: [BEAM-11589] Migrated release build.gradle file to Kotlin script

Posted by GitBox <gi...@apache.org>.
TheNeuralBit commented on a change in pull request #13948:
URL: https://github.com/apache/beam/pull/13948#discussion_r577084528



##########
File path: buildSrc/src/main/kotlin/org/apache/beam/gradle/BeamDependencies.kt
##########
@@ -0,0 +1,253 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * License); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an AS IS BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package org.apache.beam.gradle
+
+/** ***********************************************************************************************/
+// Define and export a map dependencies shared across multiple sub-projects.
+//
+// Example usage:
+// configuration {
+//   compile(library.java.avro)
+//   testCompile(library.java.junit)
+// }
+
+// These versions are defined here because they represent
+// a dependency version which should match across multiple
+// Maven artifacts.
+const val activemq_version = "5.14.5"
+const val autovalue_version = "1.7.2"
+const val aws_java_sdk_version = "1.11.718"
+const val aws_java_sdk2_version = "2.13.54"
+const val cassandra_driver_version = "3.10.2"
+const val checkerframework_version = "3.7.0"
+const val classgraph_version = "4.8.65"
+const val google_clients_version = "1.30.10"
+const val google_cloud_bigdataoss_version = "2.1.6"
+const val google_cloud_pubsub_version = "1.108.6"
+const val google_cloud_pubsublite_version = "0.7.0"
+const val google_code_gson_version = "2.8.6"
+const val google_oauth_clients_version = "1.31.0"
+// Try to keep grpc_version consistent with gRPC version in google_cloud_platform_libraries_bom
+const val grpc_version = "1.32.2"
+const val guava_version = "25.1-jre"
+const val hadoop_version = "2.10.1"
+const val hamcrest_version = "2.1"
+const val influxdb_version = "2.19"
+const val httpclient_version = "4.5.10"
+const val httpcore_version = "4.4.12"
+const val jackson_version = "2.10.2"
+const val jaxb_api_version = "2.3.3"
+const val jsr305_version = "3.0.2"
+const val kafka_version = "2.4.1"
+const val nemo_version = "0.1"
+const val netty_version = "4.1.51.Final"
+const val postgres_version = "42.2.16"
+const val powermock_version = "2.0.2"
+const val protobuf_version = "3.12.0"
+const val quickcheck_version = "0.8"
+const val slf4j_version = "1.7.30"
+const val spark_version = "2.4.7"
+const val spotbugs_version = "4.0.6"
+const val testcontainers_version = "1.15.1"
+
+// A map of maps containing common libraries used per language. To use:
+// dependencies {
+//   compile(library.java.slf4j_api)
+// }
+object library {
+    object java {
+        const val activemq_amqp                               = "org.apache.activemq:activemq-amqp:$activemq_version"
+        const val activemq_broker                             = "org.apache.activemq:activemq-broker:$activemq_version"
+        const val activemq_client                             = "org.apache.activemq:activemq-client:$activemq_version"
+        const val activemq_jaas                               = "org.apache.activemq:activemq-jaas:$activemq_version"
+        const val activemq_junit                              = "org.apache.activemq.tooling:activemq-junit:$activemq_version"
+        const val activemq_kahadb_store                       = "org.apache.activemq:activemq-kahadb-store:$activemq_version"
+        const val activemq_mqtt                               = "org.apache.activemq:activemq-mqtt:$activemq_version"
+        const val antlr                                       = "org.antlr:antlr4:4.7"
+        const val antlr_runtime                               = "org.antlr:antlr4-runtime:4.7"
+        const val args4j                                      = "args4j:args4j:2.33"
+        const val avro                                        = "org.apache.avro:avro:1.8.2"
+        const val avro_tests                                  = "org.apache.avro:avro:1.8.2:tests"
+        const val aws_java_sdk_cloudwatch                     = "com.amazonaws:aws-java-sdk-cloudwatch:$aws_java_sdk_version"
+        const val aws_java_sdk_core                           = "com.amazonaws:aws-java-sdk-core:$aws_java_sdk_version"
+        const val aws_java_sdk_dynamodb                       = "com.amazonaws:aws-java-sdk-dynamodb:$aws_java_sdk_version"
+        const val aws_java_sdk_kinesis                        = "com.amazonaws:aws-java-sdk-kinesis:$aws_java_sdk_version"
+        const val aws_java_sdk_s3                             = "com.amazonaws:aws-java-sdk-s3:$aws_java_sdk_version"
+        const val aws_java_sdk_sns                            = "com.amazonaws:aws-java-sdk-sns:$aws_java_sdk_version"
+        const val aws_java_sdk_sqs                            = "com.amazonaws:aws-java-sdk-sqs:$aws_java_sdk_version"
+        const val aws_java_sdk_sts                            = "com.amazonaws:aws-java-sdk-sts:$aws_java_sdk_version"
+        const val aws_java_sdk2_apache_client                 = "software.amazon.awssdk:apache-client:$aws_java_sdk2_version"
+        const val aws_java_sdk2_auth                          = "software.amazon.awssdk:auth:$aws_java_sdk2_version"
+        const val aws_java_sdk2_cloudwatch                    = "software.amazon.awssdk:cloudwatch:$aws_java_sdk2_version"
+        const val aws_java_sdk2_dynamodb                      = "software.amazon.awssdk:dynamodb:$aws_java_sdk2_version"
+        const val aws_java_sdk2_kinesis                       = "software.amazon.awssdk:kinesis:$aws_java_sdk2_version"
+        const val aws_java_sdk2_sdk_core                      = "software.amazon.awssdk:sdk-core:$aws_java_sdk2_version"
+        const val aws_java_sdk2_sns                           = "software.amazon.awssdk:sns:$aws_java_sdk2_version"
+        const val aws_java_sdk2_sqs                           = "software.amazon.awssdk:sqs:$aws_java_sdk2_version"
+        const val bigdataoss_gcsio                            = "com.google.cloud.bigdataoss:gcsio:$google_cloud_bigdataoss_version"
+        const val bigdataoss_util                             = "com.google.cloud.bigdataoss:util:$google_cloud_bigdataoss_version"
+        const val cassandra_driver_core                       = "com.datastax.cassandra:cassandra-driver-core:$cassandra_driver_version"
+        const val cassandra_driver_mapping                    = "com.datastax.cassandra:cassandra-driver-mapping:$cassandra_driver_version"
+        const val classgraph                                  = "io.github.classgraph:classgraph:$classgraph_version"
+        const val commons_codec                               = "commons-codec:commons-codec:1.14"
+        const val commons_compress                            = "org.apache.commons:commons-compress:1.20"
+        const val commons_csv                                 = "org.apache.commons:commons-csv:1.8"
+        const val commons_io                                  = "commons-io:commons-io:2.6"
+        const val commons_lang3                               = "org.apache.commons:commons-lang3:3.9"
+        const val commons_math3                               = "org.apache.commons:commons-math3:3.6.1"
+        const val error_prone_annotations                     = "com.google.errorprone:error_prone_annotations:2.3.1"
+        const val gax                                         = "com.google.api:gax" // google_cloud_platform_libraries_bom sets versio
+        const val gax_grpc                                    = "com.google.api:gax-grpc" // google_cloud_platform_libraries_bom sets versio
+        const val google_api_client                           = "com.google.api-client:google-api-client:$google_clients_version"
+        const val google_api_client_jackson2                  = "com.google.api-client:google-api-client-jackson2:$google_clients_version"
+        const val google_api_client_java6                     = "com.google.api-client:google-api-client-java6:$google_clients_version"
+        const val google_api_common                           = "com.google.api:api-common" // google_cloud_platform_libraries_bom sets versio
+        const val google_api_services_bigquery                = "com.google.apis:google-api-services-bigquery:v2-rev20200719-$google_clients_version"
+        const val google_api_services_clouddebugger           = "com.google.apis:google-api-services-clouddebugger:v2-rev20200501-$google_clients_version"
+        const val google_api_services_cloudresourcemanager    = "com.google.apis:google-api-services-cloudresourcemanager:v1-rev20200720-$google_clients_version"
+        const val google_api_services_dataflow                = "com.google.apis:google-api-services-dataflow:v1b3-rev20200713-$google_clients_version"
+        const val google_api_services_healthcare              = "com.google.apis:google-api-services-healthcare:v1beta1-rev20200713-$google_clients_version"
+        const val google_api_services_pubsub                  = "com.google.apis:google-api-services-pubsub:v1-rev20200713-$google_clients_version"
+        const val google_api_services_storage                 = "com.google.apis:google-api-services-storage:v1-rev20200611-$google_clients_version"
+        const val google_auth_library_credentials             = "com.google.auth:google-auth-library-credentials" // google_cloud_platform_libraries_bom sets versio
+        const val google_auth_library_oauth2_http             = "com.google.auth:google-auth-library-oauth2-http" // google_cloud_platform_libraries_bom sets versio
+        const val google_cloud_bigquery                       = "com.google.cloud:google-cloud-bigquery" // google_cloud_platform_libraries_bom sets versio
+        const val google_cloud_bigquery_storage               = "com.google.cloud:google-cloud-bigquerystorage" // google_cloud_platform_libraries_bom sets versio
+        const val google_cloud_bigtable_client_core           = "com.google.cloud.bigtable:bigtable-client-core:1.16.0"
+        const val google_cloud_bigtable_emulator              = "com.google.cloud:google-cloud-bigtable-emulator:0.125.2"
+        const val google_cloud_core                           = "com.google.cloud:google-cloud-core" // google_cloud_platform_libraries_bom sets versio
+        const val google_cloud_core_grpc                      = "com.google.cloud:google-cloud-core-grpc" // google_cloud_platform_libraries_bom sets versio
+        const val google_cloud_datacatalog_v1beta1            = "com.google.cloud:google-cloud-datacatalog" // google_cloud_platform_libraries_bom sets versio
+        const val google_cloud_dataflow_java_proto_library_all= "com.google.cloud.dataflow:google-cloud-dataflow-java-proto-library-all:0.5.160304"
+        const val google_cloud_datastore_v1_proto_client      = "com.google.cloud.datastore:datastore-v1-proto-client:1.6.3"
+        const val google_cloud_pubsub                         = "com.google.cloud:google-cloud-pubsub:$google_cloud_pubsub_version"
+        const val google_cloud_pubsublite                     = "com.google.cloud:google-cloud-pubsublite:$google_cloud_pubsublite_version"
+// The GCP Libraries BOM dashboard shows the versions set by the BOM:
+// https://storage.googleapis.com/cloud-opensource-java-dashboard/com.google.cloud/libraries-bom/13.2.0/artifact_details.html
+// Update libraries-bom version on sdks/java/container/license_scripts/dep_urls_java.yaml
+        const val google_cloud_platform_libraries_bom         = "com.google.cloud:libraries-bom:13.2.0"
+        const val google_cloud_spanner                        = "com.google.cloud:google-cloud-spanner" // google_cloud_platform_libraries_bom sets versio
+        const val google_code_gson                            = "com.google.code.gson:gson:$google_code_gson_version"
+// google-http-client's version is explicitly declared for sdks/java/maven-archetypes/examples
+// This version should be in line with the one in com.google.cloud:libraries-bom.
+        const val google_http_client                          = "com.google.http-client:google-http-client" // google_cloud_platform_libraries_bom sets versio
+        const val google_http_client_apache_v2                = "com.google.http-client:google-http-client-apache-v2" // google_cloud_platform_libraries_bom sets versio
+        const val google_http_client_jackson                  = "com.google.http-client:google-http-client-jackson:1.29.2"
+        const val google_http_client_jackson2                 = "com.google.http-client:google-http-client-jackson2" // google_cloud_platform_libraries_bom sets versio
+        const val google_http_client_protobuf                 = "com.google.http-client:google-http-client-protobuf" // google_cloud_platform_libraries_bom sets versio
+        const val google_oauth_client                         = "com.google.oauth-client:google-oauth-client:$google_oauth_clients_version"
+        const val google_oauth_client_java6                   = "com.google.oauth-client:google-oauth-client-java6:$google_oauth_clients_version"
+// Don'tconst val  use grpc_all, it can cause issues in Bazel builds. Reference the gRPC libraries you need individually instead.
+        const val grpc_alts                                   = "io.grpc:grpc-alts" // google_cloud_platform_libraries_bom sets versio
+        const val grpc_api                                    = "io.grpc:grpc-api" // google_cloud_platform_libraries_bom sets versio
+        const val grpc_auth                                   = "io.grpc:grpc-auth" // google_cloud_platform_libraries_bom sets versio
+        const val grpc_context                                = "io.grpc:grpc-context" // google_cloud_platform_libraries_bom sets versio
+        const val grpc_core                                   = "io.grpc:grpc-core" // google_cloud_platform_libraries_bom sets versio
+        const val grpc_google_cloud_pubsub_v1                 = "com.google.api.grpc:grpc-google-cloud-pubsub-v1" // google_cloud_platform_libraries_bom sets versio
+        const val grpc_google_cloud_pubsublite_v1             = "com.google.api.grpc:grpc-google-cloud-pubsublite-v1:$google_cloud_pubsublite_version"
+        const val grpc_grpclb                                 = "io.grpc:grpc-grpclb" // google_cloud_platform_libraries_bom sets versio
+        const val grpc_protobuf                               = "io.grpc:grpc-protobuf" // google_cloud_platform_libraries_bom sets versio
+        const val grpc_protobuf_lite                          = "io.grpc:grpc-protobuf-lite:$grpc_version"
+        const val grpc_netty                                  = "io.grpc:grpc-netty" // google_cloud_platform_libraries_bom sets versio
+        const val grpc_netty_shaded                           = "io.grpc:grpc-netty-shaded" // google_cloud_platform_libraries_bom sets versio
+        const val grpc_stub                                   = "io.grpc:grpc-stub" // google_cloud_platform_libraries_bom sets versio
+        const val guava                                       = "com.google.guava:guava:$guava_version"
+        const val guava_testlib                               = "com.google.guava:guava-testlib:$guava_version"
+        const val hadoop_client                               = "org.apache.hadoop:hadoop-client:$hadoop_version"
+        const val hadoop_common                               = "org.apache.hadoop:hadoop-common:$hadoop_version"
+        const val hadoop_mapreduce_client_core                = "org.apache.hadoop:hadoop-mapreduce-client-core:$hadoop_version"
+        const val hadoop_minicluster                          = "org.apache.hadoop:hadoop-minicluster:$hadoop_version"
+        const val hadoop_hdfs                                 = "org.apache.hadoop:hadoop-hdfs:$hadoop_version"
+        const val hadoop_hdfs_tests                           = "org.apache.hadoop:hadoop-hdfs:$hadoop_version:tests"
+        const val hamcrest_core                               = "org.hamcrest:hamcrest-core:$hamcrest_version"
+        const val hamcrest_library                            = "org.hamcrest:hamcrest-library:$hamcrest_version"
+        const val http_client                                 = "org.apache.httpcomponents:httpclient:$httpclient_version"
+        const val http_core                                   = "org.apache.httpcomponents:httpcore:$httpcore_version"
+        const val influxdb_library                            = "org.influxdb:influxdb-java:$influxdb_version"
+        const val jackson_annotations                         = "com.fasterxml.jackson.core:jackson-annotations:$jackson_version"
+        const val jackson_jaxb_annotations                    = "com.fasterxml.jackson.module:jackson-module-jaxb-annotations:$jackson_version"
+        const val jackson_core                                = "com.fasterxml.jackson.core:jackson-core:$jackson_version"
+        const val jackson_databind                            = "com.fasterxml.jackson.core:jackson-databind:$jackson_version"
+        const val jackson_dataformat_cbor                     = "com.fasterxml.jackson.dataformat:jackson-dataformat-cbor:$jackson_version"
+        const val jackson_dataformat_csv                      = "com.fasterxml.jackson.dataformat:jackson-dataformat-csv:$jackson_version"
+        const val jackson_dataformat_xml                      = "com.fasterxml.jackson.dataformat:jackson-dataformat-xml:$jackson_version"
+        const val jackson_dataformat_yaml                     = "com.fasterxml.jackson.dataformat:jackson-dataformat-yaml:$jackson_version"
+        const val jackson_datatype_joda                       = "com.fasterxml.jackson.datatype:jackson-datatype-joda:$jackson_version"
+        const val jackson_module_scala                        = "com.fasterxml.jackson.module:jackson-module-scala_2.11:$jackson_version"
+        const val jaxb_api                                    = "jakarta.xml.bind:jakarta.xml.bind-api:$jaxb_api_version"
+        const val jaxb_impl                                   = "com.sun.xml.bind:jaxb-impl:$jaxb_api_version"
+        const val joda_time                                   = "joda-time:joda-time:2.10.5"
+        const val jsonassert                                  = "org.skyscreamer:jsonassert:1.5.0"
+        const val jsr305                                      = "com.google.code.findbugs:jsr305:$jsr305_version"
+        const val junit                                       = "junit:junit:4.13.1"
+        const val kafka                                       = "org.apache.kafka:kafka_2.11:$kafka_version"
+        const val kafka_clients                               = "org.apache.kafka:kafka-clients:$kafka_version"
+        const val mockito_core                                = "org.mockito:mockito-core:3.7.7"
+        const val nemo_compiler_frontend_beam                 = "org.apache.nemo:nemo-compiler-frontend-beam:$nemo_version"
+        const val netty_all                                   = "io.netty:netty-all:$netty_version"
+        const val netty_handler                               = "io.netty:netty-handler:$netty_version"
+        const val netty_tcnative_boringssl_static             = "io.netty:netty-tcnative-boringssl-static:2.0.33.Final"
+        const val netty_transport_native_epoll                = "io.netty:netty-transport-native-epoll:$netty_version"
+        const val postgres                                    = "org.postgresql:postgresql:$postgres_version"
+        const val powermock                                   = "org.powermock:powermock-module-junit4:$powermock_version"
+        const val powermock_mockito                           = "org.powermock:powermock-api-mockito2:$powermock_version"
+        const val protobuf_java                               = "com.google.protobuf:protobuf-java:$protobuf_version"
+        const val protobuf_java_util                          = "com.google.protobuf:protobuf-java-util:$protobuf_version"
+        const val proto_google_cloud_bigquery_storage_v1beta1 = "com.google.api.grpc:proto-google-cloud-bigquerystorage-v1beta1" // google_cloud_platform_libraries_bom sets versio
+        const val proto_google_cloud_bigtable_v2              = "com.google.api.grpc:proto-google-cloud-bigtable-v2" // google_cloud_platform_libraries_bom sets versio
+        const val proto_google_cloud_datastore_v1             = "com.google.api.grpc:proto-google-cloud-datastore-v1" // google_cloud_platform_libraries_bom sets versio
+        const val proto_google_cloud_pubsub_v1                = "com.google.api.grpc:proto-google-cloud-pubsub-v1" // google_cloud_platform_libraries_bom sets versio
+        const val proto_google_cloud_pubsublite_v1            = "com.google.api.grpc:proto-google-cloud-pubsublite-v1:$google_cloud_pubsublite_version"
+        const val proto_google_cloud_spanner_admin_database_v1= "com.google.api.grpc:proto-google-cloud-spanner-admin-database-v1" // google_cloud_platform_libraries_bom sets versio
+        const val proto_google_common_protos                  = "com.google.api.grpc:proto-google-common-protos" // google_cloud_platform_libraries_bom sets versio
+        const val slf4j_api                                   = "org.slf4j:slf4j-api:$slf4j_version"
+        const val slf4j_simple                                = "org.slf4j:slf4j-simple:$slf4j_version"
+        const val slf4j_jdk14                                 = "org.slf4j:slf4j-jdk14:$slf4j_version"
+        const val slf4j_log4j12                               = "org.slf4j:slf4j-log4j12:$slf4j_version"
+        const val snappy_java                                 = "org.xerial.snappy:snappy-java:1.1.4"
+        const val spark_core                                  = "org.apache.spark:spark-core_2.11:$spark_version"
+        const val spark_network_common                        = "org.apache.spark:spark-network-common_2.11:$spark_version"
+        const val spark_sql                                   = "org.apache.spark:spark-sql_2.11:$spark_version"
+        const val spark_streaming                             = "org.apache.spark:spark-streaming_2.11:$spark_version"
+        const val stax2_api                                   = "org.codehaus.woodstox:stax2-api:4.2.1"
+        const val testcontainers_clickhouse                   = "org.testcontainers:clickhouse:$testcontainers_version"
+        const val testcontainers_elasticsearch                = "org.testcontainers:elasticsearch:$testcontainers_version"
+        const val testcontainers_kafka                        = "org.testcontainers:kafka:$testcontainers_version"
+        const val testcontainers_localstack                   = "org.testcontainers:localstack:$testcontainers_version"
+        const val testcontainers_postgresql                   = "org.testcontainers:postgresql:$testcontainers_version"
+        const val testcontainers_gcloud                       = "org.testcontainers:gcloud:$testcontainers_version"
+        const val vendored_bytebuddy_1_10_8                   = "org.apache.beam:beam-vendor-bytebuddy-1_10_8:0.1"
+        const val vendored_grpc_1_26_0                        = "org.apache.beam:beam-vendor-grpc-1_26_0:0.3"
+        const val vendored_guava_26_0_jre                     = "org.apache.beam:beam-vendor-guava-26_0-jre:0.1"
+        const val vendored_calcite_1_20_0                     = "org.apache.beam:beam-vendor-calcite-1_20_0:0.1"
+        const val woodstox_core_asl                           = "org.codehaus.woodstox:woodstox-core-asl:4.4.1"
+        const val zstd_jni                                    = "com.github.luben:zstd-jni:1.4.5-2"
+        const val quickcheck_core                             = "com.pholser:junit-quickcheck-core:$quickcheck_version"
+    }
+    object groovy {
+        const val groovy_all = "org.codehaus.groovy:groovy-all:2.4.13"
+    }
+    object maven {
+        const val maven_compiler_plugin = "maven-plugins:maven-compiler-plugin:3.7.0"
+        const val maven_exec_plugin     = "maven-plugins:maven-exec-plugin:1.6.0"
+        const val maven_jar_plugin      = "maven-plugins:maven-jar-plugin:3.0.2"
+        const val maven_shade_plugin    = "maven-plugins:maven-shade-plugin:3.1.0"
+        const val maven_surefire_plugin = "maven-plugins:maven-surefire-plugin:3.0.0-M5"
+    }

Review comment:
       ok thanks, sorry about that.




----------------------------------------------------------------
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 #13948: [BEAM-11589] Migrated release build.gradle file to Kotlin script

Posted by GitBox <gi...@apache.org>.
codecov[bot] edited a comment on pull request #13948:
URL: https://github.com/apache/beam/pull/13948#issuecomment-776801277


   # [Codecov](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=h1) Report
   > Merging [#13948](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=desc) (c94abc6) into [master](https://codecov.io/gh/apache/beam/commit/9aa9c8625ad3a1908dd300c4b7b949fc51828466?el=desc) (9aa9c86) will **increase** coverage by `0.00%`.
   > The diff coverage is `n/a`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/beam/pull/13948/graphs/tree.svg?width=650&height=150&src=pr&token=qcbbAh8Fj1)](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=tree)
   
   ```diff
   @@           Coverage Diff           @@
   ##           master   #13948   +/-   ##
   =======================================
     Coverage   82.94%   82.95%           
   =======================================
     Files         469      469           
     Lines       58309    58343   +34     
   =======================================
   + Hits        48367    48399   +32     
   - Misses       9942     9944    +2     
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=tree) | Coverage Δ | |
   |---|---|---|
   | [sdks/python/apache\_beam/internal/metrics/metric.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vaW50ZXJuYWwvbWV0cmljcy9tZXRyaWMucHk=) | `86.45% <0.00%> (-1.05%)` | :arrow_down: |
   | [...hon/apache\_beam/runners/worker/bundle\_processor.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy93b3JrZXIvYnVuZGxlX3Byb2Nlc3Nvci5weQ==) | `93.49% <0.00%> (-0.51%)` | :arrow_down: |
   | [...nners/portability/fn\_api\_runner/worker\_handlers.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9wb3J0YWJpbGl0eS9mbl9hcGlfcnVubmVyL3dvcmtlcl9oYW5kbGVycy5weQ==) | `80.00% <0.00%> (-0.36%)` | :arrow_down: |
   | [...\_beam/runners/portability/sdk\_container\_builder.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9wb3J0YWJpbGl0eS9zZGtfY29udGFpbmVyX2J1aWxkZXIucHk=) | `40.00% <0.00%> (-0.22%)` | :arrow_down: |
   | [...thon/apache\_beam/io/aws/clients/s3/boto3\_client.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vaW8vYXdzL2NsaWVudHMvczMvYm90bzNfY2xpZW50LnB5) | `13.22% <0.00%> (-0.12%)` | :arrow_down: |
   | [sdks/python/apache\_beam/io/iobase.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vaW8vaW9iYXNlLnB5) | `84.81% <0.00%> (+0.26%)` | :arrow_up: |
   | [...s/python/apache\_beam/runners/portability/stager.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9wb3J0YWJpbGl0eS9zdGFnZXIucHk=) | `84.19% <0.00%> (+1.56%)` | :arrow_up: |
   | [.../python/apache\_beam/transforms/periodicsequence.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vdHJhbnNmb3Jtcy9wZXJpb2RpY3NlcXVlbmNlLnB5) | `98.24% <0.00%> (+1.75%)` | :arrow_up: |
   | [...ks/python/apache\_beam/runners/worker/data\_plane.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy93b3JrZXIvZGF0YV9wbGFuZS5weQ==) | `91.31% <0.00%> (+1.79%)` | :arrow_up: |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/beam/pull/13948?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/13948?src=pr&el=footer). Last update [9aa9c86...c94abc6](https://codecov.io/gh/apache/beam/pull/13948?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 #13948: [BEAM-11589] Migrated release build.gradle file to Kotlin script

Posted by GitBox <gi...@apache.org>.
codecov[bot] edited a comment on pull request #13948:
URL: https://github.com/apache/beam/pull/13948#issuecomment-776801277


   # [Codecov](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=h1) Report
   > Merging [#13948](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=desc) (c94abc6) into [master](https://codecov.io/gh/apache/beam/commit/9aa9c8625ad3a1908dd300c4b7b949fc51828466?el=desc) (9aa9c86) will **increase** coverage by `0.00%`.
   > The diff coverage is `n/a`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/beam/pull/13948/graphs/tree.svg?width=650&height=150&src=pr&token=qcbbAh8Fj1)](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=tree)
   
   ```diff
   @@           Coverage Diff           @@
   ##           master   #13948   +/-   ##
   =======================================
     Coverage   82.94%   82.95%           
   =======================================
     Files         469      469           
     Lines       58309    58343   +34     
   =======================================
   + Hits        48367    48399   +32     
   - Misses       9942     9944    +2     
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=tree) | Coverage Δ | |
   |---|---|---|
   | [sdks/python/apache\_beam/internal/metrics/metric.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vaW50ZXJuYWwvbWV0cmljcy9tZXRyaWMucHk=) | `86.45% <0.00%> (-1.05%)` | :arrow_down: |
   | [...hon/apache\_beam/runners/worker/bundle\_processor.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy93b3JrZXIvYnVuZGxlX3Byb2Nlc3Nvci5weQ==) | `93.49% <0.00%> (-0.51%)` | :arrow_down: |
   | [...nners/portability/fn\_api\_runner/worker\_handlers.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9wb3J0YWJpbGl0eS9mbl9hcGlfcnVubmVyL3dvcmtlcl9oYW5kbGVycy5weQ==) | `80.00% <0.00%> (-0.36%)` | :arrow_down: |
   | [...\_beam/runners/portability/sdk\_container\_builder.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9wb3J0YWJpbGl0eS9zZGtfY29udGFpbmVyX2J1aWxkZXIucHk=) | `40.00% <0.00%> (-0.22%)` | :arrow_down: |
   | [...thon/apache\_beam/io/aws/clients/s3/boto3\_client.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vaW8vYXdzL2NsaWVudHMvczMvYm90bzNfY2xpZW50LnB5) | `13.22% <0.00%> (-0.12%)` | :arrow_down: |
   | [sdks/python/apache\_beam/io/iobase.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vaW8vaW9iYXNlLnB5) | `84.81% <0.00%> (+0.26%)` | :arrow_up: |
   | [...s/python/apache\_beam/runners/portability/stager.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9wb3J0YWJpbGl0eS9zdGFnZXIucHk=) | `84.19% <0.00%> (+1.56%)` | :arrow_up: |
   | [.../python/apache\_beam/transforms/periodicsequence.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vdHJhbnNmb3Jtcy9wZXJpb2RpY3NlcXVlbmNlLnB5) | `98.24% <0.00%> (+1.75%)` | :arrow_up: |
   | [...ks/python/apache\_beam/runners/worker/data\_plane.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy93b3JrZXIvZGF0YV9wbGFuZS5weQ==) | `91.31% <0.00%> (+1.79%)` | :arrow_up: |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/beam/pull/13948?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/13948?src=pr&el=footer). Last update [9aa9c86...c94abc6](https://codecov.io/gh/apache/beam/pull/13948?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] ramazan-yapparov commented on pull request #13948: [BEAM-11589] Migrated release build.gradle file to Kotlin script

Posted by GitBox <gi...@apache.org>.
ramazan-yapparov commented on pull request #13948:
URL: https://github.com/apache/beam/pull/13948#issuecomment-777237755


   > LGTM, thank you!
   > 
   > > I can move the libraries map from Groovy module to Kotlin file in buildSrc project and change is from being a Map to object
   > > That way we will be able to access that object without this ugly casts and use object fields access instead of some "magic string"
   > 
   > Are you saying you can do this once BeamModulePlugin.groovy has been converted? If so please add a TODO to remind us to follow up with that
   
   No, I can do it now - move only `libraries` and leave the rest as is


----------------------------------------------------------------
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 #13948: [BEAM-11589] Migrated release build.gradle file to Kotlin script

Posted by GitBox <gi...@apache.org>.
codecov[bot] edited a comment on pull request #13948:
URL: https://github.com/apache/beam/pull/13948#issuecomment-776801277


   # [Codecov](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=h1) Report
   > Merging [#13948](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=desc) (c94abc6) into [master](https://codecov.io/gh/apache/beam/commit/9aa9c8625ad3a1908dd300c4b7b949fc51828466?el=desc) (9aa9c86) will **increase** coverage by `0.00%`.
   > The diff coverage is `n/a`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/beam/pull/13948/graphs/tree.svg?width=650&height=150&src=pr&token=qcbbAh8Fj1)](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=tree)
   
   ```diff
   @@           Coverage Diff           @@
   ##           master   #13948   +/-   ##
   =======================================
     Coverage   82.94%   82.95%           
   =======================================
     Files         469      469           
     Lines       58309    58343   +34     
   =======================================
   + Hits        48367    48399   +32     
   - Misses       9942     9944    +2     
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=tree) | Coverage Δ | |
   |---|---|---|
   | [sdks/python/apache\_beam/internal/metrics/metric.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vaW50ZXJuYWwvbWV0cmljcy9tZXRyaWMucHk=) | `86.45% <0.00%> (-1.05%)` | :arrow_down: |
   | [...hon/apache\_beam/runners/worker/bundle\_processor.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy93b3JrZXIvYnVuZGxlX3Byb2Nlc3Nvci5weQ==) | `93.49% <0.00%> (-0.51%)` | :arrow_down: |
   | [...nners/portability/fn\_api\_runner/worker\_handlers.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9wb3J0YWJpbGl0eS9mbl9hcGlfcnVubmVyL3dvcmtlcl9oYW5kbGVycy5weQ==) | `80.00% <0.00%> (-0.36%)` | :arrow_down: |
   | [...\_beam/runners/portability/sdk\_container\_builder.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9wb3J0YWJpbGl0eS9zZGtfY29udGFpbmVyX2J1aWxkZXIucHk=) | `40.00% <0.00%> (-0.22%)` | :arrow_down: |
   | [...thon/apache\_beam/io/aws/clients/s3/boto3\_client.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vaW8vYXdzL2NsaWVudHMvczMvYm90bzNfY2xpZW50LnB5) | `13.22% <0.00%> (-0.12%)` | :arrow_down: |
   | [sdks/python/apache\_beam/io/iobase.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vaW8vaW9iYXNlLnB5) | `84.81% <0.00%> (+0.26%)` | :arrow_up: |
   | [...s/python/apache\_beam/runners/portability/stager.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9wb3J0YWJpbGl0eS9zdGFnZXIucHk=) | `84.19% <0.00%> (+1.56%)` | :arrow_up: |
   | [.../python/apache\_beam/transforms/periodicsequence.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vdHJhbnNmb3Jtcy9wZXJpb2RpY3NlcXVlbmNlLnB5) | `98.24% <0.00%> (+1.75%)` | :arrow_up: |
   | [...ks/python/apache\_beam/runners/worker/data\_plane.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy93b3JrZXIvZGF0YV9wbGFuZS5weQ==) | `91.31% <0.00%> (+1.79%)` | :arrow_up: |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/beam/pull/13948?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/13948?src=pr&el=footer). Last update [9aa9c86...c94abc6](https://codecov.io/gh/apache/beam/pull/13948?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] ramazan-yapparov commented on pull request #13948: [BEAM-11589] Migrated release build.gradle file to Kotlin script

Posted by GitBox <gi...@apache.org>.
ramazan-yapparov commented on pull request #13948:
URL: https://github.com/apache/beam/pull/13948#issuecomment-776779869


   R: @TheNeuralBit


----------------------------------------------------------------
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 #13948: [BEAM-11589] Migrated release build.gradle file to Kotlin script

Posted by GitBox <gi...@apache.org>.
codecov[bot] edited a comment on pull request #13948:
URL: https://github.com/apache/beam/pull/13948#issuecomment-776801277


   # [Codecov](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=h1) Report
   > Merging [#13948](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=desc) (022c794) into [master](https://codecov.io/gh/apache/beam/commit/827c9387d7b7efa55717d6dccde672787c72b3de?el=desc) (827c938) will **increase** coverage by `0.01%`.
   > The diff coverage is `100.00%`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/beam/pull/13948/graphs/tree.svg?width=650&height=150&src=pr&token=qcbbAh8Fj1)](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=tree)
   
   ```diff
   @@            Coverage Diff             @@
   ##           master   #13948      +/-   ##
   ==========================================
   + Coverage   82.85%   82.87%   +0.01%     
   ==========================================
     Files         466      466              
     Lines       57596    57613      +17     
   ==========================================
   + Hits        47723    47744      +21     
   + Misses       9873     9869       -4     
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=tree) | Coverage Δ | |
   |---|---|---|
   | [sdks/python/apache\_beam/dataframe/schemas.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vZGF0YWZyYW1lL3NjaGVtYXMucHk=) | `96.87% <100.00%> (+1.56%)` | :arrow_up: |
   | [sdks/python/apache\_beam/internal/metrics/metric.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vaW50ZXJuYWwvbWV0cmljcy9tZXRyaWMucHk=) | `86.45% <0.00%> (-1.05%)` | :arrow_down: |
   | [sdks/python/apache\_beam/runners/common.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9jb21tb24ucHk=) | `88.72% <0.00%> (-0.15%)` | :arrow_down: |
   | [sdks/python/apache\_beam/dataframe/expressions.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vZGF0YWZyYW1lL2V4cHJlc3Npb25zLnB5) | `91.02% <0.00%> (-0.14%)` | :arrow_down: |
   | [.../runners/portability/fn\_api\_runner/translations.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9wb3J0YWJpbGl0eS9mbl9hcGlfcnVubmVyL3RyYW5zbGF0aW9ucy5weQ==) | `92.10% <0.00%> (-0.08%)` | :arrow_down: |
   | [...on/apache\_beam/runners/dataflow/dataflow\_runner.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9kYXRhZmxvdy9kYXRhZmxvd19ydW5uZXIucHk=) | `82.28% <0.00%> (ø)` | |
   | [...apache\_beam/runners/portability/portable\_runner.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9wb3J0YWJpbGl0eS9wb3J0YWJsZV9ydW5uZXIucHk=) | `76.30% <0.00%> (ø)` | |
   | [...eam/runners/portability/fn\_api\_runner/fn\_runner.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9wb3J0YWJpbGl0eS9mbl9hcGlfcnVubmVyL2ZuX3J1bm5lci5weQ==) | `89.87% <0.00%> (ø)` | |
   | [sdks/python/apache\_beam/dataframe/frames.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vZGF0YWZyYW1lL2ZyYW1lcy5weQ==) | `91.07% <0.00%> (+0.13%)` | :arrow_up: |
   | [...ks/python/apache\_beam/runners/worker/sdk\_worker.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy93b3JrZXIvc2RrX3dvcmtlci5weQ==) | `89.69% <0.00%> (+0.15%)` | :arrow_up: |
   | ... and [3 more](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree-more) | |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/beam/pull/13948?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/13948?src=pr&el=footer). Last update [befcc3d...022c794](https://codecov.io/gh/apache/beam/pull/13948?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] ramazan-yapparov commented on a change in pull request #13948: [BEAM-11589] Migrated release build.gradle file to Kotlin script

Posted by GitBox <gi...@apache.org>.
ramazan-yapparov commented on a change in pull request #13948:
URL: https://github.com/apache/beam/pull/13948#discussion_r576034326



##########
File path: buildSrc/build.gradle.kts
##########
@@ -21,6 +21,7 @@ plugins {
   `java-gradle-plugin`
   groovy
   id("com.diffplug.spotless") version "5.6.1"
+  `kotlin-dsl`

Review comment:
       It wasn't
   I added this dependency because I moved dependencies to Kotlin in `buildSrc` module




----------------------------------------------------------------
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 #13948: [BEAM-11589] Migrated release build.gradle file to Kotlin script

Posted by GitBox <gi...@apache.org>.
codecov[bot] edited a comment on pull request #13948:
URL: https://github.com/apache/beam/pull/13948#issuecomment-776801277


   # [Codecov](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=h1) Report
   > Merging [#13948](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=desc) (c8f3b51) into [master](https://codecov.io/gh/apache/beam/commit/9aa9c8625ad3a1908dd300c4b7b949fc51828466?el=desc) (9aa9c86) will **not change** coverage.
   > The diff coverage is `n/a`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/beam/pull/13948/graphs/tree.svg?width=650&height=150&src=pr&token=qcbbAh8Fj1)](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=tree)
   
   ```diff
   @@           Coverage Diff           @@
   ##           master   #13948   +/-   ##
   =======================================
     Coverage   82.94%   82.94%           
   =======================================
     Files         469      469           
     Lines       58309    58309           
   =======================================
     Hits        48367    48367           
     Misses       9942     9942           
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=tree) | Coverage Δ | |
   |---|---|---|
   | [sdks/python/apache\_beam/internal/metrics/metric.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vaW50ZXJuYWwvbWV0cmljcy9tZXRyaWMucHk=) | `86.45% <0.00%> (-1.05%)` | :arrow_down: |
   | [...nners/portability/fn\_api\_runner/worker\_handlers.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9wb3J0YWJpbGl0eS9mbl9hcGlfcnVubmVyL3dvcmtlcl9oYW5kbGVycy5weQ==) | `80.00% <0.00%> (-0.36%)` | :arrow_down: |
   | [...hon/apache\_beam/runners/worker/bundle\_processor.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy93b3JrZXIvYnVuZGxlX3Byb2Nlc3Nvci5weQ==) | `93.86% <0.00%> (-0.13%)` | :arrow_down: |
   | [.../python/apache\_beam/transforms/periodicsequence.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vdHJhbnNmb3Jtcy9wZXJpb2RpY3NlcXVlbmNlLnB5) | `98.24% <0.00%> (+1.75%)` | :arrow_up: |
   | [...hon/apache\_beam/runners/direct/test\_stream\_impl.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9kaXJlY3QvdGVzdF9zdHJlYW1faW1wbC5weQ==) | `94.11% <0.00%> (+2.20%)` | :arrow_up: |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/beam/pull/13948?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/13948?src=pr&el=footer). Last update [9aa9c86...c8f3b51](https://codecov.io/gh/apache/beam/pull/13948?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 #13948: [BEAM-11589] Migrated release build.gradle file to Kotlin script

Posted by GitBox <gi...@apache.org>.
codecov[bot] edited a comment on pull request #13948:
URL: https://github.com/apache/beam/pull/13948#issuecomment-776801277


   # [Codecov](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=h1) Report
   > Merging [#13948](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=desc) (c8f3b51) into [master](https://codecov.io/gh/apache/beam/commit/9aa9c8625ad3a1908dd300c4b7b949fc51828466?el=desc) (9aa9c86) will **not change** coverage.
   > The diff coverage is `n/a`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/beam/pull/13948/graphs/tree.svg?width=650&height=150&src=pr&token=qcbbAh8Fj1)](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=tree)
   
   ```diff
   @@           Coverage Diff           @@
   ##           master   #13948   +/-   ##
   =======================================
     Coverage   82.94%   82.94%           
   =======================================
     Files         469      469           
     Lines       58309    58309           
   =======================================
     Hits        48367    48367           
     Misses       9942     9942           
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=tree) | Coverage Δ | |
   |---|---|---|
   | [sdks/python/apache\_beam/internal/metrics/metric.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vaW50ZXJuYWwvbWV0cmljcy9tZXRyaWMucHk=) | `86.45% <0.00%> (-1.05%)` | :arrow_down: |
   | [...nners/portability/fn\_api\_runner/worker\_handlers.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9wb3J0YWJpbGl0eS9mbl9hcGlfcnVubmVyL3dvcmtlcl9oYW5kbGVycy5weQ==) | `80.00% <0.00%> (-0.36%)` | :arrow_down: |
   | [...hon/apache\_beam/runners/worker/bundle\_processor.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy93b3JrZXIvYnVuZGxlX3Byb2Nlc3Nvci5weQ==) | `93.86% <0.00%> (-0.13%)` | :arrow_down: |
   | [.../python/apache\_beam/transforms/periodicsequence.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vdHJhbnNmb3Jtcy9wZXJpb2RpY3NlcXVlbmNlLnB5) | `98.24% <0.00%> (+1.75%)` | :arrow_up: |
   | [...hon/apache\_beam/runners/direct/test\_stream\_impl.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9kaXJlY3QvdGVzdF9zdHJlYW1faW1wbC5weQ==) | `94.11% <0.00%> (+2.20%)` | :arrow_up: |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/beam/pull/13948?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/13948?src=pr&el=footer). Last update [9aa9c86...c8f3b51](https://codecov.io/gh/apache/beam/pull/13948?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] ramazan-yapparov commented on a change in pull request #13948: [BEAM-11589] Migrated release build.gradle file to Kotlin script

Posted by GitBox <gi...@apache.org>.
ramazan-yapparov commented on a change in pull request #13948:
URL: https://github.com/apache/beam/pull/13948#discussion_r576033934



##########
File path: buildSrc/src/main/kotlin/org/apache/beam/gradle/BeamDependencies.kt
##########
@@ -0,0 +1,253 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * License); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an AS IS BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package org.apache.beam.gradle
+
+/** ***********************************************************************************************/
+// Define and export a map dependencies shared across multiple sub-projects.
+//
+// Example usage:
+// configuration {
+//   compile(library.java.avro)
+//   testCompile(library.java.junit)
+// }
+
+// These versions are defined here because they represent
+// a dependency version which should match across multiple
+// Maven artifacts.
+const val activemq_version = "5.14.5"
+const val autovalue_version = "1.7.2"
+const val aws_java_sdk_version = "1.11.718"
+const val aws_java_sdk2_version = "2.13.54"
+const val cassandra_driver_version = "3.10.2"
+const val checkerframework_version = "3.7.0"
+const val classgraph_version = "4.8.65"
+const val google_clients_version = "1.30.10"
+const val google_cloud_bigdataoss_version = "2.1.6"
+const val google_cloud_pubsub_version = "1.108.6"
+const val google_cloud_pubsublite_version = "0.7.0"
+const val google_code_gson_version = "2.8.6"
+const val google_oauth_clients_version = "1.31.0"
+// Try to keep grpc_version consistent with gRPC version in google_cloud_platform_libraries_bom
+const val grpc_version = "1.32.2"
+const val guava_version = "25.1-jre"
+const val hadoop_version = "2.10.1"
+const val hamcrest_version = "2.1"
+const val influxdb_version = "2.19"
+const val httpclient_version = "4.5.10"
+const val httpcore_version = "4.4.12"
+const val jackson_version = "2.10.2"
+const val jaxb_api_version = "2.3.3"
+const val jsr305_version = "3.0.2"
+const val kafka_version = "2.4.1"
+const val nemo_version = "0.1"
+const val netty_version = "4.1.51.Final"
+const val postgres_version = "42.2.16"
+const val powermock_version = "2.0.2"
+const val protobuf_version = "3.12.0"
+const val quickcheck_version = "0.8"
+const val slf4j_version = "1.7.30"
+const val spark_version = "2.4.7"
+const val spotbugs_version = "4.0.6"
+const val testcontainers_version = "1.15.1"
+
+// A map of maps containing common libraries used per language. To use:
+// dependencies {
+//   compile(library.java.slf4j_api)
+// }
+object library {
+    object java {
+        const val activemq_amqp                               = "org.apache.activemq:activemq-amqp:$activemq_version"
+        const val activemq_broker                             = "org.apache.activemq:activemq-broker:$activemq_version"
+        const val activemq_client                             = "org.apache.activemq:activemq-client:$activemq_version"
+        const val activemq_jaas                               = "org.apache.activemq:activemq-jaas:$activemq_version"
+        const val activemq_junit                              = "org.apache.activemq.tooling:activemq-junit:$activemq_version"
+        const val activemq_kahadb_store                       = "org.apache.activemq:activemq-kahadb-store:$activemq_version"
+        const val activemq_mqtt                               = "org.apache.activemq:activemq-mqtt:$activemq_version"
+        const val antlr                                       = "org.antlr:antlr4:4.7"
+        const val antlr_runtime                               = "org.antlr:antlr4-runtime:4.7"
+        const val args4j                                      = "args4j:args4j:2.33"
+        const val avro                                        = "org.apache.avro:avro:1.8.2"
+        const val avro_tests                                  = "org.apache.avro:avro:1.8.2:tests"
+        const val aws_java_sdk_cloudwatch                     = "com.amazonaws:aws-java-sdk-cloudwatch:$aws_java_sdk_version"
+        const val aws_java_sdk_core                           = "com.amazonaws:aws-java-sdk-core:$aws_java_sdk_version"
+        const val aws_java_sdk_dynamodb                       = "com.amazonaws:aws-java-sdk-dynamodb:$aws_java_sdk_version"
+        const val aws_java_sdk_kinesis                        = "com.amazonaws:aws-java-sdk-kinesis:$aws_java_sdk_version"
+        const val aws_java_sdk_s3                             = "com.amazonaws:aws-java-sdk-s3:$aws_java_sdk_version"
+        const val aws_java_sdk_sns                            = "com.amazonaws:aws-java-sdk-sns:$aws_java_sdk_version"
+        const val aws_java_sdk_sqs                            = "com.amazonaws:aws-java-sdk-sqs:$aws_java_sdk_version"
+        const val aws_java_sdk_sts                            = "com.amazonaws:aws-java-sdk-sts:$aws_java_sdk_version"
+        const val aws_java_sdk2_apache_client                 = "software.amazon.awssdk:apache-client:$aws_java_sdk2_version"
+        const val aws_java_sdk2_auth                          = "software.amazon.awssdk:auth:$aws_java_sdk2_version"
+        const val aws_java_sdk2_cloudwatch                    = "software.amazon.awssdk:cloudwatch:$aws_java_sdk2_version"
+        const val aws_java_sdk2_dynamodb                      = "software.amazon.awssdk:dynamodb:$aws_java_sdk2_version"
+        const val aws_java_sdk2_kinesis                       = "software.amazon.awssdk:kinesis:$aws_java_sdk2_version"
+        const val aws_java_sdk2_sdk_core                      = "software.amazon.awssdk:sdk-core:$aws_java_sdk2_version"
+        const val aws_java_sdk2_sns                           = "software.amazon.awssdk:sns:$aws_java_sdk2_version"
+        const val aws_java_sdk2_sqs                           = "software.amazon.awssdk:sqs:$aws_java_sdk2_version"
+        const val bigdataoss_gcsio                            = "com.google.cloud.bigdataoss:gcsio:$google_cloud_bigdataoss_version"
+        const val bigdataoss_util                             = "com.google.cloud.bigdataoss:util:$google_cloud_bigdataoss_version"
+        const val cassandra_driver_core                       = "com.datastax.cassandra:cassandra-driver-core:$cassandra_driver_version"
+        const val cassandra_driver_mapping                    = "com.datastax.cassandra:cassandra-driver-mapping:$cassandra_driver_version"
+        const val classgraph                                  = "io.github.classgraph:classgraph:$classgraph_version"
+        const val commons_codec                               = "commons-codec:commons-codec:1.14"
+        const val commons_compress                            = "org.apache.commons:commons-compress:1.20"
+        const val commons_csv                                 = "org.apache.commons:commons-csv:1.8"
+        const val commons_io                                  = "commons-io:commons-io:2.6"
+        const val commons_lang3                               = "org.apache.commons:commons-lang3:3.9"
+        const val commons_math3                               = "org.apache.commons:commons-math3:3.6.1"
+        const val error_prone_annotations                     = "com.google.errorprone:error_prone_annotations:2.3.1"
+        const val gax                                         = "com.google.api:gax" // google_cloud_platform_libraries_bom sets versio
+        const val gax_grpc                                    = "com.google.api:gax-grpc" // google_cloud_platform_libraries_bom sets versio
+        const val google_api_client                           = "com.google.api-client:google-api-client:$google_clients_version"
+        const val google_api_client_jackson2                  = "com.google.api-client:google-api-client-jackson2:$google_clients_version"
+        const val google_api_client_java6                     = "com.google.api-client:google-api-client-java6:$google_clients_version"
+        const val google_api_common                           = "com.google.api:api-common" // google_cloud_platform_libraries_bom sets versio
+        const val google_api_services_bigquery                = "com.google.apis:google-api-services-bigquery:v2-rev20200719-$google_clients_version"
+        const val google_api_services_clouddebugger           = "com.google.apis:google-api-services-clouddebugger:v2-rev20200501-$google_clients_version"
+        const val google_api_services_cloudresourcemanager    = "com.google.apis:google-api-services-cloudresourcemanager:v1-rev20200720-$google_clients_version"
+        const val google_api_services_dataflow                = "com.google.apis:google-api-services-dataflow:v1b3-rev20200713-$google_clients_version"
+        const val google_api_services_healthcare              = "com.google.apis:google-api-services-healthcare:v1beta1-rev20200713-$google_clients_version"
+        const val google_api_services_pubsub                  = "com.google.apis:google-api-services-pubsub:v1-rev20200713-$google_clients_version"
+        const val google_api_services_storage                 = "com.google.apis:google-api-services-storage:v1-rev20200611-$google_clients_version"
+        const val google_auth_library_credentials             = "com.google.auth:google-auth-library-credentials" // google_cloud_platform_libraries_bom sets versio
+        const val google_auth_library_oauth2_http             = "com.google.auth:google-auth-library-oauth2-http" // google_cloud_platform_libraries_bom sets versio
+        const val google_cloud_bigquery                       = "com.google.cloud:google-cloud-bigquery" // google_cloud_platform_libraries_bom sets versio
+        const val google_cloud_bigquery_storage               = "com.google.cloud:google-cloud-bigquerystorage" // google_cloud_platform_libraries_bom sets versio
+        const val google_cloud_bigtable_client_core           = "com.google.cloud.bigtable:bigtable-client-core:1.16.0"
+        const val google_cloud_bigtable_emulator              = "com.google.cloud:google-cloud-bigtable-emulator:0.125.2"
+        const val google_cloud_core                           = "com.google.cloud:google-cloud-core" // google_cloud_platform_libraries_bom sets versio
+        const val google_cloud_core_grpc                      = "com.google.cloud:google-cloud-core-grpc" // google_cloud_platform_libraries_bom sets versio
+        const val google_cloud_datacatalog_v1beta1            = "com.google.cloud:google-cloud-datacatalog" // google_cloud_platform_libraries_bom sets versio
+        const val google_cloud_dataflow_java_proto_library_all= "com.google.cloud.dataflow:google-cloud-dataflow-java-proto-library-all:0.5.160304"
+        const val google_cloud_datastore_v1_proto_client      = "com.google.cloud.datastore:datastore-v1-proto-client:1.6.3"
+        const val google_cloud_pubsub                         = "com.google.cloud:google-cloud-pubsub:$google_cloud_pubsub_version"
+        const val google_cloud_pubsublite                     = "com.google.cloud:google-cloud-pubsublite:$google_cloud_pubsublite_version"
+// The GCP Libraries BOM dashboard shows the versions set by the BOM:
+// https://storage.googleapis.com/cloud-opensource-java-dashboard/com.google.cloud/libraries-bom/13.2.0/artifact_details.html
+// Update libraries-bom version on sdks/java/container/license_scripts/dep_urls_java.yaml
+        const val google_cloud_platform_libraries_bom         = "com.google.cloud:libraries-bom:13.2.0"
+        const val google_cloud_spanner                        = "com.google.cloud:google-cloud-spanner" // google_cloud_platform_libraries_bom sets versio
+        const val google_code_gson                            = "com.google.code.gson:gson:$google_code_gson_version"
+// google-http-client's version is explicitly declared for sdks/java/maven-archetypes/examples
+// This version should be in line with the one in com.google.cloud:libraries-bom.
+        const val google_http_client                          = "com.google.http-client:google-http-client" // google_cloud_platform_libraries_bom sets versio
+        const val google_http_client_apache_v2                = "com.google.http-client:google-http-client-apache-v2" // google_cloud_platform_libraries_bom sets versio
+        const val google_http_client_jackson                  = "com.google.http-client:google-http-client-jackson:1.29.2"
+        const val google_http_client_jackson2                 = "com.google.http-client:google-http-client-jackson2" // google_cloud_platform_libraries_bom sets versio
+        const val google_http_client_protobuf                 = "com.google.http-client:google-http-client-protobuf" // google_cloud_platform_libraries_bom sets versio
+        const val google_oauth_client                         = "com.google.oauth-client:google-oauth-client:$google_oauth_clients_version"
+        const val google_oauth_client_java6                   = "com.google.oauth-client:google-oauth-client-java6:$google_oauth_clients_version"
+// Don'tconst val  use grpc_all, it can cause issues in Bazel builds. Reference the gRPC libraries you need individually instead.
+        const val grpc_alts                                   = "io.grpc:grpc-alts" // google_cloud_platform_libraries_bom sets versio
+        const val grpc_api                                    = "io.grpc:grpc-api" // google_cloud_platform_libraries_bom sets versio
+        const val grpc_auth                                   = "io.grpc:grpc-auth" // google_cloud_platform_libraries_bom sets versio
+        const val grpc_context                                = "io.grpc:grpc-context" // google_cloud_platform_libraries_bom sets versio
+        const val grpc_core                                   = "io.grpc:grpc-core" // google_cloud_platform_libraries_bom sets versio
+        const val grpc_google_cloud_pubsub_v1                 = "com.google.api.grpc:grpc-google-cloud-pubsub-v1" // google_cloud_platform_libraries_bom sets versio
+        const val grpc_google_cloud_pubsublite_v1             = "com.google.api.grpc:grpc-google-cloud-pubsublite-v1:$google_cloud_pubsublite_version"
+        const val grpc_grpclb                                 = "io.grpc:grpc-grpclb" // google_cloud_platform_libraries_bom sets versio
+        const val grpc_protobuf                               = "io.grpc:grpc-protobuf" // google_cloud_platform_libraries_bom sets versio
+        const val grpc_protobuf_lite                          = "io.grpc:grpc-protobuf-lite:$grpc_version"
+        const val grpc_netty                                  = "io.grpc:grpc-netty" // google_cloud_platform_libraries_bom sets versio
+        const val grpc_netty_shaded                           = "io.grpc:grpc-netty-shaded" // google_cloud_platform_libraries_bom sets versio
+        const val grpc_stub                                   = "io.grpc:grpc-stub" // google_cloud_platform_libraries_bom sets versio
+        const val guava                                       = "com.google.guava:guava:$guava_version"
+        const val guava_testlib                               = "com.google.guava:guava-testlib:$guava_version"
+        const val hadoop_client                               = "org.apache.hadoop:hadoop-client:$hadoop_version"
+        const val hadoop_common                               = "org.apache.hadoop:hadoop-common:$hadoop_version"
+        const val hadoop_mapreduce_client_core                = "org.apache.hadoop:hadoop-mapreduce-client-core:$hadoop_version"
+        const val hadoop_minicluster                          = "org.apache.hadoop:hadoop-minicluster:$hadoop_version"
+        const val hadoop_hdfs                                 = "org.apache.hadoop:hadoop-hdfs:$hadoop_version"
+        const val hadoop_hdfs_tests                           = "org.apache.hadoop:hadoop-hdfs:$hadoop_version:tests"
+        const val hamcrest_core                               = "org.hamcrest:hamcrest-core:$hamcrest_version"
+        const val hamcrest_library                            = "org.hamcrest:hamcrest-library:$hamcrest_version"
+        const val http_client                                 = "org.apache.httpcomponents:httpclient:$httpclient_version"
+        const val http_core                                   = "org.apache.httpcomponents:httpcore:$httpcore_version"
+        const val influxdb_library                            = "org.influxdb:influxdb-java:$influxdb_version"
+        const val jackson_annotations                         = "com.fasterxml.jackson.core:jackson-annotations:$jackson_version"
+        const val jackson_jaxb_annotations                    = "com.fasterxml.jackson.module:jackson-module-jaxb-annotations:$jackson_version"
+        const val jackson_core                                = "com.fasterxml.jackson.core:jackson-core:$jackson_version"
+        const val jackson_databind                            = "com.fasterxml.jackson.core:jackson-databind:$jackson_version"
+        const val jackson_dataformat_cbor                     = "com.fasterxml.jackson.dataformat:jackson-dataformat-cbor:$jackson_version"
+        const val jackson_dataformat_csv                      = "com.fasterxml.jackson.dataformat:jackson-dataformat-csv:$jackson_version"
+        const val jackson_dataformat_xml                      = "com.fasterxml.jackson.dataformat:jackson-dataformat-xml:$jackson_version"
+        const val jackson_dataformat_yaml                     = "com.fasterxml.jackson.dataformat:jackson-dataformat-yaml:$jackson_version"
+        const val jackson_datatype_joda                       = "com.fasterxml.jackson.datatype:jackson-datatype-joda:$jackson_version"
+        const val jackson_module_scala                        = "com.fasterxml.jackson.module:jackson-module-scala_2.11:$jackson_version"
+        const val jaxb_api                                    = "jakarta.xml.bind:jakarta.xml.bind-api:$jaxb_api_version"
+        const val jaxb_impl                                   = "com.sun.xml.bind:jaxb-impl:$jaxb_api_version"
+        const val joda_time                                   = "joda-time:joda-time:2.10.5"
+        const val jsonassert                                  = "org.skyscreamer:jsonassert:1.5.0"
+        const val jsr305                                      = "com.google.code.findbugs:jsr305:$jsr305_version"
+        const val junit                                       = "junit:junit:4.13.1"
+        const val kafka                                       = "org.apache.kafka:kafka_2.11:$kafka_version"
+        const val kafka_clients                               = "org.apache.kafka:kafka-clients:$kafka_version"
+        const val mockito_core                                = "org.mockito:mockito-core:3.7.7"
+        const val nemo_compiler_frontend_beam                 = "org.apache.nemo:nemo-compiler-frontend-beam:$nemo_version"
+        const val netty_all                                   = "io.netty:netty-all:$netty_version"
+        const val netty_handler                               = "io.netty:netty-handler:$netty_version"
+        const val netty_tcnative_boringssl_static             = "io.netty:netty-tcnative-boringssl-static:2.0.33.Final"
+        const val netty_transport_native_epoll                = "io.netty:netty-transport-native-epoll:$netty_version"
+        const val postgres                                    = "org.postgresql:postgresql:$postgres_version"
+        const val powermock                                   = "org.powermock:powermock-module-junit4:$powermock_version"
+        const val powermock_mockito                           = "org.powermock:powermock-api-mockito2:$powermock_version"
+        const val protobuf_java                               = "com.google.protobuf:protobuf-java:$protobuf_version"
+        const val protobuf_java_util                          = "com.google.protobuf:protobuf-java-util:$protobuf_version"
+        const val proto_google_cloud_bigquery_storage_v1beta1 = "com.google.api.grpc:proto-google-cloud-bigquerystorage-v1beta1" // google_cloud_platform_libraries_bom sets versio
+        const val proto_google_cloud_bigtable_v2              = "com.google.api.grpc:proto-google-cloud-bigtable-v2" // google_cloud_platform_libraries_bom sets versio
+        const val proto_google_cloud_datastore_v1             = "com.google.api.grpc:proto-google-cloud-datastore-v1" // google_cloud_platform_libraries_bom sets versio
+        const val proto_google_cloud_pubsub_v1                = "com.google.api.grpc:proto-google-cloud-pubsub-v1" // google_cloud_platform_libraries_bom sets versio
+        const val proto_google_cloud_pubsublite_v1            = "com.google.api.grpc:proto-google-cloud-pubsublite-v1:$google_cloud_pubsublite_version"
+        const val proto_google_cloud_spanner_admin_database_v1= "com.google.api.grpc:proto-google-cloud-spanner-admin-database-v1" // google_cloud_platform_libraries_bom sets versio
+        const val proto_google_common_protos                  = "com.google.api.grpc:proto-google-common-protos" // google_cloud_platform_libraries_bom sets versio
+        const val slf4j_api                                   = "org.slf4j:slf4j-api:$slf4j_version"
+        const val slf4j_simple                                = "org.slf4j:slf4j-simple:$slf4j_version"
+        const val slf4j_jdk14                                 = "org.slf4j:slf4j-jdk14:$slf4j_version"
+        const val slf4j_log4j12                               = "org.slf4j:slf4j-log4j12:$slf4j_version"
+        const val snappy_java                                 = "org.xerial.snappy:snappy-java:1.1.4"
+        const val spark_core                                  = "org.apache.spark:spark-core_2.11:$spark_version"
+        const val spark_network_common                        = "org.apache.spark:spark-network-common_2.11:$spark_version"
+        const val spark_sql                                   = "org.apache.spark:spark-sql_2.11:$spark_version"
+        const val spark_streaming                             = "org.apache.spark:spark-streaming_2.11:$spark_version"
+        const val stax2_api                                   = "org.codehaus.woodstox:stax2-api:4.2.1"
+        const val testcontainers_clickhouse                   = "org.testcontainers:clickhouse:$testcontainers_version"
+        const val testcontainers_elasticsearch                = "org.testcontainers:elasticsearch:$testcontainers_version"
+        const val testcontainers_kafka                        = "org.testcontainers:kafka:$testcontainers_version"
+        const val testcontainers_localstack                   = "org.testcontainers:localstack:$testcontainers_version"
+        const val testcontainers_postgresql                   = "org.testcontainers:postgresql:$testcontainers_version"
+        const val testcontainers_gcloud                       = "org.testcontainers:gcloud:$testcontainers_version"
+        const val vendored_bytebuddy_1_10_8                   = "org.apache.beam:beam-vendor-bytebuddy-1_10_8:0.1"
+        const val vendored_grpc_1_26_0                        = "org.apache.beam:beam-vendor-grpc-1_26_0:0.3"
+        const val vendored_guava_26_0_jre                     = "org.apache.beam:beam-vendor-guava-26_0-jre:0.1"
+        const val vendored_calcite_1_20_0                     = "org.apache.beam:beam-vendor-calcite-1_20_0:0.1"
+        const val woodstox_core_asl                           = "org.codehaus.woodstox:woodstox-core-asl:4.4.1"
+        const val zstd_jni                                    = "com.github.luben:zstd-jni:1.4.5-2"
+        const val quickcheck_core                             = "com.pholser:junit-quickcheck-core:$quickcheck_version"
+    }
+    object groovy {
+        const val groovy_all = "org.codehaus.groovy:groovy-all:2.4.13"
+    }
+    object maven {
+        const val maven_compiler_plugin = "maven-plugins:maven-compiler-plugin:3.7.0"
+        const val maven_exec_plugin     = "maven-plugins:maven-exec-plugin:1.6.0"
+        const val maven_jar_plugin      = "maven-plugins:maven-jar-plugin:3.0.2"
+        const val maven_shade_plugin    = "maven-plugins:maven-shade-plugin:3.1.0"
+        const val maven_surefire_plugin = "maven-plugins:maven-surefire-plugin:3.0.0-M5"
+    }

Review comment:
       Right now the only way is to duplicate the definitions.
   I guess we should move dependencies declaration to Kotlin at the same time as moving `BeamModulePlugin`




----------------------------------------------------------------
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] ramazan-yapparov commented on pull request #13948: [BEAM-11589] Migrated release build.gradle file to Kotlin script

Posted by GitBox <gi...@apache.org>.
ramazan-yapparov commented on pull request #13948:
URL: https://github.com/apache/beam/pull/13948#issuecomment-776784369


   @TheNeuralBit 
   I can move the `libraries` map from Groovy module to Kotlin file in `buildSrc` project and change is from being a `Map` to `object`
   That way we will be able to access that object without this ugly casts and use object fields access instead of some "magic string"
   
   https://github.com/apache/beam/blob/022c7943df9817347bca6a6ed74cef90711ba47e/release/build.gradle.kts#L28-L33


----------------------------------------------------------------
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 #13948: [BEAM-11589] Migrated release build.gradle file to Kotlin script

Posted by GitBox <gi...@apache.org>.
codecov[bot] edited a comment on pull request #13948:
URL: https://github.com/apache/beam/pull/13948#issuecomment-776801277


   # [Codecov](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=h1) Report
   > Merging [#13948](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=desc) (022c794) into [master](https://codecov.io/gh/apache/beam/commit/827c9387d7b7efa55717d6dccde672787c72b3de?el=desc) (827c938) will **increase** coverage by `0.00%`.
   > The diff coverage is `100.00%`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/beam/pull/13948/graphs/tree.svg?width=650&height=150&src=pr&token=qcbbAh8Fj1)](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=tree)
   
   ```diff
   @@           Coverage Diff           @@
   ##           master   #13948   +/-   ##
   =======================================
     Coverage   82.85%   82.86%           
   =======================================
     Files         466      466           
     Lines       57596    57613   +17     
   =======================================
   + Hits        47723    47742   +19     
   + Misses       9873     9871    -2     
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=tree) | Coverage Δ | |
   |---|---|---|
   | [sdks/python/apache\_beam/dataframe/schemas.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vZGF0YWZyYW1lL3NjaGVtYXMucHk=) | `96.87% <100.00%> (+1.56%)` | :arrow_up: |
   | [sdks/python/apache\_beam/internal/metrics/metric.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vaW50ZXJuYWwvbWV0cmljcy9tZXRyaWMucHk=) | `86.45% <0.00%> (-1.05%)` | :arrow_down: |
   | [sdks/python/apache\_beam/runners/common.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9jb21tb24ucHk=) | `88.72% <0.00%> (-0.15%)` | :arrow_down: |
   | [sdks/python/apache\_beam/dataframe/expressions.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vZGF0YWZyYW1lL2V4cHJlc3Npb25zLnB5) | `91.02% <0.00%> (-0.14%)` | :arrow_down: |
   | [.../runners/portability/fn\_api\_runner/translations.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9wb3J0YWJpbGl0eS9mbl9hcGlfcnVubmVyL3RyYW5zbGF0aW9ucy5weQ==) | `92.10% <0.00%> (-0.08%)` | :arrow_down: |
   | [...on/apache\_beam/runners/dataflow/dataflow\_runner.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9kYXRhZmxvdy9kYXRhZmxvd19ydW5uZXIucHk=) | `82.28% <0.00%> (ø)` | |
   | [...apache\_beam/runners/portability/portable\_runner.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9wb3J0YWJpbGl0eS9wb3J0YWJsZV9ydW5uZXIucHk=) | `76.30% <0.00%> (ø)` | |
   | [...eam/runners/portability/fn\_api\_runner/fn\_runner.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9wb3J0YWJpbGl0eS9mbl9hcGlfcnVubmVyL2ZuX3J1bm5lci5weQ==) | `89.87% <0.00%> (ø)` | |
   | [sdks/python/apache\_beam/dataframe/frames.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vZGF0YWZyYW1lL2ZyYW1lcy5weQ==) | `91.07% <0.00%> (+0.13%)` | :arrow_up: |
   | [...ks/python/apache\_beam/runners/worker/sdk\_worker.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy93b3JrZXIvc2RrX3dvcmtlci5weQ==) | `89.69% <0.00%> (+0.15%)` | :arrow_up: |
   | ... and [2 more](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree-more) | |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/beam/pull/13948?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/13948?src=pr&el=footer). Last update [befcc3d...022c794](https://codecov.io/gh/apache/beam/pull/13948?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 #13948: [BEAM-11589] Migrated release build.gradle file to Kotlin script

Posted by GitBox <gi...@apache.org>.
codecov[bot] edited a comment on pull request #13948:
URL: https://github.com/apache/beam/pull/13948#issuecomment-776801277


   # [Codecov](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=h1) Report
   > Merging [#13948](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=desc) (022c794) into [master](https://codecov.io/gh/apache/beam/commit/9aa9c8625ad3a1908dd300c4b7b949fc51828466?el=desc) (9aa9c86) will **decrease** coverage by `0.07%`.
   > The diff coverage is `n/a`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/beam/pull/13948/graphs/tree.svg?width=650&height=150&src=pr&token=qcbbAh8Fj1)](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=tree)
   
   ```diff
   @@            Coverage Diff             @@
   ##           master   #13948      +/-   ##
   ==========================================
   - Coverage   82.94%   82.87%   -0.08%     
   ==========================================
     Files         469      466       -3     
     Lines       58309    57613     -696     
   ==========================================
   - Hits        48367    47744     -623     
   + Misses       9942     9869      -73     
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=tree) | Coverage Δ | |
   |---|---|---|
   | [sdks/python/apache\_beam/coders/slow\_stream.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vY29kZXJzL3Nsb3dfc3RyZWFtLnB5) | `92.43% <0.00%> (-2.21%)` | :arrow_down: |
   | [sdks/python/apache\_beam/dataframe/frame\_base.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vZGF0YWZyYW1lL2ZyYW1lX2Jhc2UucHk=) | `88.46% <0.00%> (-2.09%)` | :arrow_down: |
   | [...eam/runners/portability/fn\_api\_runner/execution.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9wb3J0YWJpbGl0eS9mbl9hcGlfcnVubmVyL2V4ZWN1dGlvbi5weQ==) | `92.03% <0.00%> (-1.65%)` | :arrow_down: |
   | [sdks/python/apache\_beam/internal/metrics/metric.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vaW50ZXJuYWwvbWV0cmljcy9tZXRyaWMucHk=) | `86.45% <0.00%> (-1.05%)` | :arrow_down: |
   | [...dks/python/apache\_beam/runners/pipeline\_context.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9waXBlbGluZV9jb250ZXh0LnB5) | `92.74% <0.00%> (-0.81%)` | :arrow_down: |
   | [sdks/python/apache\_beam/internal/pickler.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vaW50ZXJuYWwvcGlja2xlci5weQ==) | `86.33% <0.00%> (-0.54%)` | :arrow_down: |
   | [...ks/python/apache\_beam/coders/coders\_test\_common.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vY29kZXJzL2NvZGVyc190ZXN0X2NvbW1vbi5weQ==) | `98.52% <0.00%> (-0.36%)` | :arrow_down: |
   | [...nners/portability/fn\_api\_runner/worker\_handlers.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9wb3J0YWJpbGl0eS9mbl9hcGlfcnVubmVyL3dvcmtlcl9oYW5kbGVycy5weQ==) | `80.00% <0.00%> (-0.36%)` | :arrow_down: |
   | [sdks/python/apache\_beam/coders/coders.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vY29kZXJzL2NvZGVycy5weQ==) | `87.26% <0.00%> (-0.22%)` | :arrow_down: |
   | [sdks/python/apache\_beam/coders/coder\_impl.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vY29kZXJzL2NvZGVyX2ltcGwucHk=) | `95.38% <0.00%> (-0.14%)` | :arrow_down: |
   | ... and [11 more](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree-more) | |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/beam/pull/13948?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/13948?src=pr&el=footer). Last update [9aa9c86...c8f3b51](https://codecov.io/gh/apache/beam/pull/13948?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] ramazan-yapparov commented on pull request #13948: [BEAM-11589] Migrated release build.gradle file to Kotlin script

Posted by GitBox <gi...@apache.org>.
ramazan-yapparov commented on pull request #13948:
URL: https://github.com/apache/beam/pull/13948#issuecomment-777065481


   Run Java PreCommit


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [beam] codecov[bot] edited a comment on pull request #13948: [BEAM-11589] Migrated release build.gradle file to Kotlin script

Posted by GitBox <gi...@apache.org>.
codecov[bot] edited a comment on pull request #13948:
URL: https://github.com/apache/beam/pull/13948#issuecomment-776801277


   # [Codecov](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=h1) Report
   > Merging [#13948](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=desc) (c8f3b51) into [master](https://codecov.io/gh/apache/beam/commit/9aa9c8625ad3a1908dd300c4b7b949fc51828466?el=desc) (9aa9c86) will **not change** coverage.
   > The diff coverage is `n/a`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/beam/pull/13948/graphs/tree.svg?width=650&height=150&src=pr&token=qcbbAh8Fj1)](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=tree)
   
   ```diff
   @@           Coverage Diff           @@
   ##           master   #13948   +/-   ##
   =======================================
     Coverage   82.94%   82.94%           
   =======================================
     Files         469      469           
     Lines       58309    58309           
   =======================================
     Hits        48367    48367           
     Misses       9942     9942           
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=tree) | Coverage Δ | |
   |---|---|---|
   | [sdks/python/apache\_beam/internal/metrics/metric.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vaW50ZXJuYWwvbWV0cmljcy9tZXRyaWMucHk=) | `86.45% <0.00%> (-1.05%)` | :arrow_down: |
   | [...nners/portability/fn\_api\_runner/worker\_handlers.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9wb3J0YWJpbGl0eS9mbl9hcGlfcnVubmVyL3dvcmtlcl9oYW5kbGVycy5weQ==) | `80.00% <0.00%> (-0.36%)` | :arrow_down: |
   | [...hon/apache\_beam/runners/worker/bundle\_processor.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy93b3JrZXIvYnVuZGxlX3Byb2Nlc3Nvci5weQ==) | `93.86% <0.00%> (-0.13%)` | :arrow_down: |
   | [.../python/apache\_beam/transforms/periodicsequence.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vdHJhbnNmb3Jtcy9wZXJpb2RpY3NlcXVlbmNlLnB5) | `98.24% <0.00%> (+1.75%)` | :arrow_up: |
   | [...hon/apache\_beam/runners/direct/test\_stream\_impl.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9kaXJlY3QvdGVzdF9zdHJlYW1faW1wbC5weQ==) | `94.11% <0.00%> (+2.20%)` | :arrow_up: |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/beam/pull/13948?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/13948?src=pr&el=footer). Last update [9aa9c86...c8f3b51](https://codecov.io/gh/apache/beam/pull/13948?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 #13948: [BEAM-11589] Migrated release build.gradle file to Kotlin script

Posted by GitBox <gi...@apache.org>.
codecov[bot] edited a comment on pull request #13948:
URL: https://github.com/apache/beam/pull/13948#issuecomment-776801277


   # [Codecov](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=h1) Report
   > Merging [#13948](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=desc) (c94abc6) into [master](https://codecov.io/gh/apache/beam/commit/9aa9c8625ad3a1908dd300c4b7b949fc51828466?el=desc) (9aa9c86) will **increase** coverage by `0.00%`.
   > The diff coverage is `n/a`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/beam/pull/13948/graphs/tree.svg?width=650&height=150&src=pr&token=qcbbAh8Fj1)](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=tree)
   
   ```diff
   @@           Coverage Diff           @@
   ##           master   #13948   +/-   ##
   =======================================
     Coverage   82.94%   82.95%           
   =======================================
     Files         469      469           
     Lines       58309    58343   +34     
   =======================================
   + Hits        48367    48399   +32     
   - Misses       9942     9944    +2     
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=tree) | Coverage Δ | |
   |---|---|---|
   | [sdks/python/apache\_beam/internal/metrics/metric.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vaW50ZXJuYWwvbWV0cmljcy9tZXRyaWMucHk=) | `86.45% <0.00%> (-1.05%)` | :arrow_down: |
   | [...hon/apache\_beam/runners/worker/bundle\_processor.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy93b3JrZXIvYnVuZGxlX3Byb2Nlc3Nvci5weQ==) | `93.49% <0.00%> (-0.51%)` | :arrow_down: |
   | [...nners/portability/fn\_api\_runner/worker\_handlers.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9wb3J0YWJpbGl0eS9mbl9hcGlfcnVubmVyL3dvcmtlcl9oYW5kbGVycy5weQ==) | `80.00% <0.00%> (-0.36%)` | :arrow_down: |
   | [...\_beam/runners/portability/sdk\_container\_builder.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9wb3J0YWJpbGl0eS9zZGtfY29udGFpbmVyX2J1aWxkZXIucHk=) | `40.00% <0.00%> (-0.22%)` | :arrow_down: |
   | [...thon/apache\_beam/io/aws/clients/s3/boto3\_client.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vaW8vYXdzL2NsaWVudHMvczMvYm90bzNfY2xpZW50LnB5) | `13.22% <0.00%> (-0.12%)` | :arrow_down: |
   | [sdks/python/apache\_beam/io/iobase.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vaW8vaW9iYXNlLnB5) | `84.81% <0.00%> (+0.26%)` | :arrow_up: |
   | [...s/python/apache\_beam/runners/portability/stager.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9wb3J0YWJpbGl0eS9zdGFnZXIucHk=) | `84.19% <0.00%> (+1.56%)` | :arrow_up: |
   | [.../python/apache\_beam/transforms/periodicsequence.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vdHJhbnNmb3Jtcy9wZXJpb2RpY3NlcXVlbmNlLnB5) | `98.24% <0.00%> (+1.75%)` | :arrow_up: |
   | [...ks/python/apache\_beam/runners/worker/data\_plane.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy93b3JrZXIvZGF0YV9wbGFuZS5weQ==) | `91.31% <0.00%> (+1.79%)` | :arrow_up: |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/beam/pull/13948?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/13948?src=pr&el=footer). Last update [9aa9c86...c94abc6](https://codecov.io/gh/apache/beam/pull/13948?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 #13948: [BEAM-11589] Migrated release build.gradle file to Kotlin script

Posted by GitBox <gi...@apache.org>.
codecov[bot] edited a comment on pull request #13948:
URL: https://github.com/apache/beam/pull/13948#issuecomment-776801277


   # [Codecov](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=h1) Report
   > Merging [#13948](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=desc) (022c794) into [master](https://codecov.io/gh/apache/beam/commit/827c9387d7b7efa55717d6dccde672787c72b3de?el=desc) (827c938) will **increase** coverage by `0.01%`.
   > The diff coverage is `100.00%`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/beam/pull/13948/graphs/tree.svg?width=650&height=150&src=pr&token=qcbbAh8Fj1)](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=tree)
   
   ```diff
   @@            Coverage Diff             @@
   ##           master   #13948      +/-   ##
   ==========================================
   + Coverage   82.85%   82.87%   +0.01%     
   ==========================================
     Files         466      466              
     Lines       57596    57613      +17     
   ==========================================
   + Hits        47723    47744      +21     
   + Misses       9873     9869       -4     
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=tree) | Coverage Δ | |
   |---|---|---|
   | [sdks/python/apache\_beam/dataframe/schemas.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vZGF0YWZyYW1lL3NjaGVtYXMucHk=) | `96.87% <100.00%> (+1.56%)` | :arrow_up: |
   | [sdks/python/apache\_beam/internal/metrics/metric.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vaW50ZXJuYWwvbWV0cmljcy9tZXRyaWMucHk=) | `86.45% <0.00%> (-1.05%)` | :arrow_down: |
   | [sdks/python/apache\_beam/runners/common.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9jb21tb24ucHk=) | `88.72% <0.00%> (-0.15%)` | :arrow_down: |
   | [sdks/python/apache\_beam/dataframe/expressions.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vZGF0YWZyYW1lL2V4cHJlc3Npb25zLnB5) | `91.02% <0.00%> (-0.14%)` | :arrow_down: |
   | [.../runners/portability/fn\_api\_runner/translations.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9wb3J0YWJpbGl0eS9mbl9hcGlfcnVubmVyL3RyYW5zbGF0aW9ucy5weQ==) | `92.10% <0.00%> (-0.08%)` | :arrow_down: |
   | [...on/apache\_beam/runners/dataflow/dataflow\_runner.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9kYXRhZmxvdy9kYXRhZmxvd19ydW5uZXIucHk=) | `82.28% <0.00%> (ø)` | |
   | [...apache\_beam/runners/portability/portable\_runner.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9wb3J0YWJpbGl0eS9wb3J0YWJsZV9ydW5uZXIucHk=) | `76.30% <0.00%> (ø)` | |
   | [...eam/runners/portability/fn\_api\_runner/fn\_runner.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9wb3J0YWJpbGl0eS9mbl9hcGlfcnVubmVyL2ZuX3J1bm5lci5weQ==) | `89.87% <0.00%> (ø)` | |
   | [sdks/python/apache\_beam/dataframe/frames.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vZGF0YWZyYW1lL2ZyYW1lcy5weQ==) | `91.07% <0.00%> (+0.13%)` | :arrow_up: |
   | [...ks/python/apache\_beam/runners/worker/sdk\_worker.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy93b3JrZXIvc2RrX3dvcmtlci5weQ==) | `89.69% <0.00%> (+0.15%)` | :arrow_up: |
   | ... and [3 more](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree-more) | |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/beam/pull/13948?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/13948?src=pr&el=footer). Last update [befcc3d...022c794](https://codecov.io/gh/apache/beam/pull/13948?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 #13948: [BEAM-11589] Migrated release build.gradle file to Kotlin script

Posted by GitBox <gi...@apache.org>.
codecov[bot] edited a comment on pull request #13948:
URL: https://github.com/apache/beam/pull/13948#issuecomment-776801277


   # [Codecov](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=h1) Report
   > Merging [#13948](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=desc) (c8f3b51) into [master](https://codecov.io/gh/apache/beam/commit/9aa9c8625ad3a1908dd300c4b7b949fc51828466?el=desc) (9aa9c86) will **not change** coverage.
   > The diff coverage is `n/a`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/beam/pull/13948/graphs/tree.svg?width=650&height=150&src=pr&token=qcbbAh8Fj1)](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=tree)
   
   ```diff
   @@           Coverage Diff           @@
   ##           master   #13948   +/-   ##
   =======================================
     Coverage   82.94%   82.94%           
   =======================================
     Files         469      469           
     Lines       58309    58309           
   =======================================
     Hits        48367    48367           
     Misses       9942     9942           
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/beam/pull/13948?src=pr&el=tree) | Coverage Δ | |
   |---|---|---|
   | [sdks/python/apache\_beam/internal/metrics/metric.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vaW50ZXJuYWwvbWV0cmljcy9tZXRyaWMucHk=) | `86.45% <0.00%> (-1.05%)` | :arrow_down: |
   | [...nners/portability/fn\_api\_runner/worker\_handlers.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9wb3J0YWJpbGl0eS9mbl9hcGlfcnVubmVyL3dvcmtlcl9oYW5kbGVycy5weQ==) | `80.00% <0.00%> (-0.36%)` | :arrow_down: |
   | [...hon/apache\_beam/runners/worker/bundle\_processor.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy93b3JrZXIvYnVuZGxlX3Byb2Nlc3Nvci5weQ==) | `93.86% <0.00%> (-0.13%)` | :arrow_down: |
   | [.../python/apache\_beam/transforms/periodicsequence.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vdHJhbnNmb3Jtcy9wZXJpb2RpY3NlcXVlbmNlLnB5) | `98.24% <0.00%> (+1.75%)` | :arrow_up: |
   | [...hon/apache\_beam/runners/direct/test\_stream\_impl.py](https://codecov.io/gh/apache/beam/pull/13948/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9kaXJlY3QvdGVzdF9zdHJlYW1faW1wbC5weQ==) | `94.11% <0.00%> (+2.20%)` | :arrow_up: |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/beam/pull/13948?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/13948?src=pr&el=footer). Last update [9aa9c86...c94abc6](https://codecov.io/gh/apache/beam/pull/13948?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