Closed Shaddoll closed 1 month ago
Attention: Patch coverage is 40.00000%
with 6 lines
in your changes are missing coverage. Please review.
Project coverage is 68.22%. Comparing base (
dd54cd0
) to head (d1869a3
). Report is 4 commits behind head on master.
Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
---|---|---|---|---|---|
service/matching/tasklist/task_list_manager.go | 1 | 2 | 50.0% | ||
service/matching/tasklist/task_reader.go | 0 | 1 | 0.0% | ||
common/persistence/persistence-tests/persistenceTestBase.go | 0 | 2 | 0.0% | ||
<!-- | Total: | 10 | 14 | 71.43% | --> |
Files with Coverage Reduction | New Missed Lines | % | ||
---|---|---|---|---|
service/history/task/transfer_standby_task_executor.go | 2 | 86.33% | ||
common/task/weighted_round_robin_task_scheduler.go | 2 | 88.56% | ||
common/task/parallel_task_processor.go | 2 | 93.06% | ||
common/task/fifo_task_scheduler.go | 2 | 80.41% | ||
common/persistence/sql/sqlplugin/mysql/task.go | 2 | 73.68% | ||
common/persistence/visibility_single_manager.go | 2 | 99.33% | ||
service/matching/tasklist/matcher.go | 2 | 89.35% | ||
common/persistence/sql/sqlplugin/mysql/db.go | 2 | 79.49% | ||
service/history/queue/timer_queue_processor_base.go | 3 | 77.87% | ||
common/persistence/wrappers/errorinjectors/utils.go | 3 | 93.7% | ||
<!-- | Total: | 1231 | --> |
Totals | |
---|---|
Change from base Build 018fcf32-23d9-4d7d-a579-94202b6c6918: | -0.03% |
Covered Lines: | 102829 |
Relevant Lines: | 147693 |
What changed? Remove redundant parameter from CreateTasks flow
Why? persistence.TaskInfo has workflowID and runID fields, which makes Execution parameter redundant.
How did you test it? unit tests
Potential risks
Release notes
Documentation Changes