Context
A TODO comment in the codebase needs to be addressed after migrating from single-container to multi-container workers.
Location
File: components/clp-package-utils/clp_package_utils/controller.py
Method: DockerComposeController._get_num_workers()
Line: ~630
TODO Comment
TODO: Revisit after moving from single-container to multi-container workers.
Current Behaviour
The _get_num_workers() method currently returns cpu_count() // 2 to determine the parallelism number for worker components. This implementation assumes a single-container deployment model.
Action Required
Once the architecture moves to multi-container workers, the worker parallelism strategy should be revisited and potentially updated to align with the new deployment model.
References
Context
A TODO comment in the codebase needs to be addressed after migrating from single-container to multi-container workers.
Location
File:
components/clp-package-utils/clp_package_utils/controller.pyMethod:
DockerComposeController._get_num_workers()Line: ~630
TODO Comment
Current Behaviour
The
_get_num_workers()method currently returnscpu_count() // 2to determine the parallelism number for worker components. This implementation assumes a single-container deployment model.Action Required
Once the architecture moves to multi-container workers, the worker parallelism strategy should be revisited and potentially updated to align with the new deployment model.
References