[ https://issues.apache.org/jira/browse/BEAM-1833?focusedWorklogId=532007&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-532007 ]
ASF GitHub Bot logged work on BEAM-1833: ---------------------------------------- Author: ASF GitHub Bot Created on: 06/Jan/21 17:31 Start Date: 06/Jan/21 17:31 Worklog Time Spent: 10m Work Description: codecov[bot] edited a comment on pull request #13672: URL: https://github.com/apache/beam/pull/13672#issuecomment-755025462 # [Codecov](https://codecov.io/gh/apache/beam/pull/13672?src=pr&el=h1) Report > Merging [#13672](https://codecov.io/gh/apache/beam/pull/13672?src=pr&el=desc) (9aefb8a) into [master](https://codecov.io/gh/apache/beam/commit/e12a0d84b2cd0fc561e89796cedbce141b3b4359?el=desc) (e12a0d8) will **increase** coverage by `0.01%`. > The diff coverage is `100.00%`. [![Impacted file tree graph](https://codecov.io/gh/apache/beam/pull/13672/graphs/tree.svg?width=650&height=150&src=pr&token=qcbbAh8Fj1)](https://codecov.io/gh/apache/beam/pull/13672?src=pr&el=tree) ```diff @@ Coverage Diff @@ ## master #13672 +/- ## ========================================== + Coverage 82.73% 82.75% +0.01% ========================================== Files 466 466 Lines 57518 57521 +3 ========================================== + Hits 47589 47601 +12 + Misses 9929 9920 -9 ``` | [Impacted Files](https://codecov.io/gh/apache/beam/pull/13672?src=pr&el=tree) | Coverage Δ | | |---|---|---| | [sdks/python/apache\_beam/dataframe/frames.py](https://codecov.io/gh/apache/beam/pull/13672/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vZGF0YWZyYW1lL2ZyYW1lcy5weQ==) | `91.98% <100.00%> (ø)` | | | [sdks/python/apache\_beam/pipeline.py](https://codecov.io/gh/apache/beam/pull/13672/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcGlwZWxpbmUucHk=) | `91.13% <100.00%> (+0.80%)` | :arrow_up: | | [sdks/python/apache\_beam/transforms/ptransform.py](https://codecov.io/gh/apache/beam/pull/13672/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vdHJhbnNmb3Jtcy9wdHJhbnNmb3JtLnB5) | `93.06% <100.00%> (+0.10%)` | :arrow_up: | | [sdks/python/apache\_beam/utils/interactive\_utils.py](https://codecov.io/gh/apache/beam/pull/13672/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vdXRpbHMvaW50ZXJhY3RpdmVfdXRpbHMucHk=) | `88.09% <0.00%> (-2.39%)` | :arrow_down: | | [.../python/apache\_beam/transforms/periodicsequence.py](https://codecov.io/gh/apache/beam/pull/13672/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vdHJhbnNmb3Jtcy9wZXJpb2RpY3NlcXVlbmNlLnB5) | `98.24% <0.00%> (+1.75%)` | :arrow_up: | | [...che\_beam/runners/interactive/interactive\_runner.py](https://codecov.io/gh/apache/beam/pull/13672/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vcnVubmVycy9pbnRlcmFjdGl2ZS9pbnRlcmFjdGl2ZV9ydW5uZXIucHk=) | `92.72% <0.00%> (+1.81%)` | :arrow_up: | | [.../python/apache\_beam/testing/test\_stream\_service.py](https://codecov.io/gh/apache/beam/pull/13672/diff?src=pr&el=tree#diff-c2Rrcy9weXRob24vYXBhY2hlX2JlYW0vdGVzdGluZy90ZXN0X3N0cmVhbV9zZXJ2aWNlLnB5) | `93.18% <0.00%> (+4.54%)` | :arrow_up: | ------ [Continue to review full report at Codecov](https://codecov.io/gh/apache/beam/pull/13672?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/13672?src=pr&el=footer). Last update [9f9dcd6...93c2bd8](https://codecov.io/gh/apache/beam/pull/13672?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: us...@infra.apache.org Issue Time Tracking ------------------- Worklog Id: (was: 532007) Time Spent: 5h 10m (was: 5h) > Restructure Python pipeline construction to better follow the Runner API > ------------------------------------------------------------------------ > > Key: BEAM-1833 > URL: https://issues.apache.org/jira/browse/BEAM-1833 > Project: Beam > Issue Type: Improvement > Components: sdk-py-core > Reporter: Robert Bradshaw > Assignee: Sam Rohde > Priority: P2 > Fix For: 2.20.0 > > Time Spent: 5h 10m > Remaining Estimate: 0h > > The most important part is removing the runner.apply overrides, but there are > also various other improvements (e.g. all inputs and outputs should be named). -- This message was sent by Atlassian Jira (v8.3.4#803005)