That was easy! Got to love the rust library ecosystem.

The threads seem to default on the number of CPUs which is fine. We can always tune this later with some user feedback.

At some point, we'll need to forward the exit status to the parent process exit status if any of those fail, although it can happen at a later stage.

Thank you for helping us to implement this. It wasn't easy for me to choose the library (threadpool, rayon, crossbeam) for this issue. I've been researching which library to use while working on another issue. Have you tried crossbeam? In my research, crossbeam provides various queues that allow pushing and popping items from several threads without a mutex.

The code is LGTM. And I agree with what @zimbatm said, we can change it later if we need to.

Regarding the error, I haven't think of displaying multiple errors when the formatters run in parallel.

Have you tried crossbeam?

Yes, and it's a great library. rayon effectively uses crossbeam under the hood. I haven't tried the threadpool crate, so I can't comment on that one (though rayon does offer a very similar threadpool API if we ever need it).

I think for this use case rayon's ParallelIterator is perfect. There are no dependencies between the tasks so we don't need to use channels explicitly and it makes for a short/concise implementation.

I haven't think of displaying multiple errors

I will try to come up with something 😄 (probably a different PR)
We need to think of where we want to see these errors: stdout/stderr/log file, and whether the --quiet flag affects it. We probably also want to specify which file/formatter is at the origin of the errors.

rayon effectively uses crossbeam under the hood

Oh, this is great, I didn't notice it. We should stick with rayon then.

I think for this use case rayon's ParallelIterator is perfect. There are no dependencies between the tasks so we don't need to use channels explicitly and it makes for a short/concise implementation.

Yes, I agree with this.

I will try to come up with something 😄 (probably a different PR)

Yes, agree that it should be in a different PR.