Closed Groxx closed 3 weeks ago
Attention: Patch coverage is 67.36842%
with 31 lines
in your changes missing coverage. Please review.
Project coverage is 71.99%. Comparing base (
b3e21fa
) to head (e2b0994
). Report is 1 commits behind head on master.
Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
---|---|---|---|---|---|
common/quotas/global/collection/collection.go | 86 | 128 | 67.19% | ||
<!-- | Total: | 94 | 136 | 69.12% | --> |
Files with Coverage Reduction | New Missed Lines | % | ||
---|---|---|---|---|
common/cache/lru.go | 2 | 93.01% | ||
service/history/replication/task_processor.go | 2 | 82.76% | ||
service/history/task/transfer_standby_task_executor.go | 3 | 87.55% | ||
service/history/task/task.go | 3 | 84.81% | ||
service/history/queue/timer_queue_processor_base.go | 3 | 77.87% | ||
service/history/handler/handler.go | 4 | 96.14% | ||
service/history/execution/cache.go | 6 | 74.61% | ||
<!-- | Total: | 23 | --> |
Totals | |
---|---|
Change from base Build 018feeec-d7a4-4bb0-b278-1977f0e4c7ed: | 0.04% |
Covered Lines: | 106410 |
Relevant Lines: | 149093 |
A half-fleshed-out skeleton of the quotas.Collection that the frontend will use for these limits, and a chunk of pseudo-code to show how it'll be used when the RPC PR is ready (coming soon).
This is not completely usable right now and will likely need some small changes, but it has the core bits:
It will eventually need: