Open nagisa opened 2 years ago
My gut feeling is that we should unconditionally not use rayon perhaps? It won't help with worst-case, and, architecture-wise, it is perhaps more prudent to add || on a higher level:
That would work too, sure.
Ok, let's just remove rayon then!
Related discussion: https://github.com/near/nearcore-private-1/issues/2#issuecomment-1256030324
Describe the bug
Making use of rayon optional is causing us to take suboptimal implementation decisions that hurt both rayon and non-rayon use-cases (such as collecting twice, first into a vector, then into
PrimaryMap
.)If making use of rayon unconditional (if you need to limit worker threads, setup the thread pool accordingly) makes way for simplifying those areas of code, we should just go ahead with it.