You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
mirror push: PushParams.Parallelism is unused; push is sequential #427
d8 mirror push uploads images sequentially, but PushParams.Parallelism is populated and never read — it is dead configuration. This also makes the README "Best Practices → Parallel Operations" note misleading for push.
pkg/libmirror/operations/params/push.go defines ParallelismConfig and DefaultParallelism.
No code reads .Parallelism (only the definition and the assignment reference it), and the push path contains no concurrency primitives (go func, errgroup, sync.WaitGroup). internal/mirror/pusher/pusher.go pushes in a plain sequential for loop.
Impact
Dead config; pushes of large bundles are slower than the config suggests, and documentation implying parallel push is inaccurate.
Expected
Either wire the Parallelism config into the pusher, or remove the unused config and adjust the docs.
Severity: low (cleanup / performance). As of main (62132d6).
Activity