Closed sankari165 closed 2 weeks ago
Attention: Patch coverage is 94.11765%
with 1 line
in your changes missing coverage. Please review.
Project coverage is 72.21%. Comparing base (
ecb93a1
) to head (037fb7a
).
Files with Coverage Reduction | New Missed Lines | % | ||
---|---|---|---|---|
common/task/weighted_round_robin_task_scheduler.go | 2 | 89.05% | ||
service/matching/tasklist/db.go | 2 | 73.23% | ||
common/persistence/sql/sqlplugin/mysql/task.go | 2 | 73.68% | ||
common/util.go | 2 | 91.84% | ||
service/history/execution/mutable_state_builder.go | 2 | 78.45% | ||
service/matching/tasklist/matcher.go | 2 | 89.35% | ||
common/persistence/sql/sqlplugin/mysql/db.go | 2 | 79.49% | ||
service/history/task/task.go | 3 | 84.81% | ||
common/log/tag/tags.go | 3 | 50.46% | ||
service/history/task/fetcher.go | 3 | 86.6% | ||
<!-- | Total: | 59 | --> |
Totals | |
---|---|
Change from base Build 019008cc-dc35-4c24-a6bc-044c76fa1f43: | 0.01% |
Covered Lines: | 106532 |
Relevant Lines: | 149110 |
What changed? Sorting was by default set to StartTime in Descending Order. This PR enables custom sorting
Why? It would help to provide this as feature enhancement in cadence-web v4
How did you test it? unit tests
Potential risks Fallback to default behaviour prevents risk
Release notes
Documentation Changes