1. Matt Chaput
  2. whoosh
  3. Issues
Issue #367 new

FileIndex.writer with procs=0/None/False should create MpWriter

Daniel Black
created an issue

because MpWriter uses a false value to create procs=cpu_count()

Comments (2)

  1. Matt Chaput repo owner

    It's a mistake that the code in MpWriter that calls cpu_count() is essentially unreachable, but I don't think it makes intuitive sense to have procs=0/None/False use multiple processes.

    I think the right thing for now is for the user to call multiprocessing.cpu_count() themselves and pass the value to the procs= keyword argument.

  2. Log in to comment