[ 
https://issues.apache.org/jira/browse/BEAM-14469?focusedWorklogId=770339&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-770339
 ]

ASF GitHub Bot logged work on BEAM-14469:
-----------------------------------------

                Author: ASF GitHub Bot
            Created on: 13/May/22 19:32
            Start Date: 13/May/22 19:32
    Worklog Time Spent: 10m 
      Work Description: codecov[bot] commented on PR #17667:
URL: https://github.com/apache/beam/pull/17667#issuecomment-1126378911

   # 
[Codecov](https://codecov.io/gh/apache/beam/pull/17667?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
 Report
   > Merging 
[#17667](https://codecov.io/gh/apache/beam/pull/17667?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
 (26a3043) into 
[master](https://codecov.io/gh/apache/beam/commit/787479f1a5e178333ded3ff02331163c4fe75f1a?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
 (787479f) will **increase** coverage by `0.00%`.
   > The diff coverage is `100.00%`.
   
   ```diff
   @@           Coverage Diff           @@
   ##           master   #17667   +/-   ##
   =======================================
     Coverage   73.97%   73.97%           
   =======================================
     Files         693      693           
     Lines       91724    91727    +3     
   =======================================
   + Hits        67853    67856    +3     
     Misses      22622    22622           
     Partials     1249     1249           
   ```
   
   | Flag | Coverage Δ | |
   |---|---|---|
   | go | `50.37% <100.00%> (+<0.01%)` | :arrow_up: |
   
   Flags with carried forward coverage won't be shown. [Click 
here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#carryforward-flags-in-the-pull-request-comment)
 to find out more.
   
   | [Impacted 
Files](https://codecov.io/gh/apache/beam/pull/17667?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
 | Coverage Δ | |
   |---|---|---|
   | 
[sdks/go/pkg/beam/core/runtime/exec/sdf.go](https://codecov.io/gh/apache/beam/pull/17667/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-c2Rrcy9nby9wa2cvYmVhbS9jb3JlL3J1bnRpbWUvZXhlYy9zZGYuZ28=)
 | `71.01% <100.00%> (+0.18%)` | :arrow_up: |
   
   ------
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/beam/pull/17667?src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
   > **Legend** - [Click here to learn 
more](https://docs.codecov.io/docs/codecov-delta?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
   > `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
   > Powered by 
[Codecov](https://codecov.io/gh/apache/beam/pull/17667?src=pr&el=footer&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
 Last update 
[787479f...26a3043](https://codecov.io/gh/apache/beam/pull/17667?src=pr&el=lastupdated&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
 Read the [comment 
docs](https://docs.codecov.io/docs/pull-request-comments?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
   




Issue Time Tracking
-------------------

    Worklog Id:     (was: 770339)
    Time Spent: 3h 20m  (was: 3h 10m)

> TestCheckpointing produces too many zero-length restrictions
> ------------------------------------------------------------
>
>                 Key: BEAM-14469
>                 URL: https://issues.apache.org/jira/browse/BEAM-14469
>             Project: Beam
>          Issue Type: Improvement
>          Components: sdk-go
>            Reporter: Jack McCluskey
>            Assignee: Jack McCluskey
>            Priority: P2
>          Time Spent: 3h 20m
>  Remaining Estimate: 0h
>
> The TestCheckpointing integration test produces a large number of zero-length 
> restrictions on split, leading to extremely verbose logging and errors when 
> run on dataflow/with multiple runners. 
>  
> This is tied to the offsetrange tracker producing zero-length restrictions on 
> TrySplit() calls in edge cases. The unintended usage of the tracker in this 
> streaming context creates this issue but did reveal that TrySplit() calls 
> were not allowing nil primary returns, which could cause issues moving 
> forward. 



--
This message was sent by Atlassian Jira
(v8.20.7#820007)

Reply via email to