[MINOR] Extend LaunchedWorkerInfo + Remove 'numSectors'
MERGE REQUEST
Overview
This MR adds a potential read job error to the appropriate launched worker info, for debug purposes.
It also removes numSectors
and replaces it with a TODO. I think it's not terribly high priority to tackle it but I wanted to mention it rather than just remove the unused numSectors
parameter. In general we should probably make queue estimates more generic, allowing queues to track on tags or something. This would also help with read jobs and the read sizes. Currently we track 3 separate fixed sizes.
Example for Visual Changes
N/A
Checklist
Review and complete the checklist to ensure that the MR is complete before assigned to an approver.
-
All new methods or updated methods have clear docstrings -
Testing added or updated for new methods -
Any new packages are added to Makefile and .gitlab-ci.yml -
API documentation updated for API updates -
Module README.md updated for changes to workflow -
Issue added to Sia-UI repo for new supporting features -
Changelog File Created
Issues Closed
Closes #4502 (closed)
Closes #4496 (closed)
Closes #4376 (closed)
Edited by Peter-Jan Brone