Update job_status to report the closest integer percentage of test cases completed instead of the weird pseudo-percentage used by test_progress.
We could support both at the same time by:
- having the database store progress in the range 0–9900 inclusive,
- having
job_status return n / 99, and
- having
test_progress return if created { 0 } else if running { max(1, n/100) } else { 100 }.
Alternatives
- We could let
job_status round its percentage to the closest integer instead of always rounding down.
- We could let
job_status keep reporting weird pseudo-percentages until we remove test_progress, and then update job_status to report proper percentages.
- We could leave
job_status as is and let the meaning of the progress percentage remain vague.
- We could leave
job_status as is and just document its progress values in all their weridness.
Stabilization
Tracking issue
Affected interfaces
Compatibly changed:
Dependencies
Overlapping proposals
None
Update
job_statusto report the closest integer percentage of test cases completed instead of the weird pseudo-percentage used bytest_progress.We could support both at the same time by:
job_statusreturnn / 99, andtest_progressreturnif created { 0 } else if running { max(1, n/100) } else { 100 }.Alternatives
job_statusround its percentage to the closest integer instead of always rounding down.job_statuskeep reporting weird pseudo-percentages until we removetest_progress, and then updatejob_statusto report proper percentages.job_statusas is and let the meaning of theprogress percentageremain vague.job_statusas is and just document its progress values in all their weridness.Stabilization
Tracking issue
Affected interfaces
Compatibly changed:
Dependencies
Overlapping proposals
None