Use at least 2 threads for usher-sampled

yatisht / usher

Ultrafast Sample Placement on Existing Trees

MIT License

121 stars 40 forks source link

Closed yceh closed 1 year ago

yceh commented 1 year ago

Usher-sampled needs one thread for tree modifications and at least another one for search. If -T1 is specified, it will deadlock.

yatisht commented 1 year ago

I can approve this now, but in general, I don't think it's a good practice to disallow single threaded execution.