Skip to content

Revert the multiprocessing change from #214.

Philip Elson requested to merge pelson/cara:fix/revert-multiprocessing into master

I haven't been able to fully understand the reason for the error, which looks like:

ERROR UUID 2c4dbefb-16f8-4370-b9a2-cc92a51e9bae
Traceback (most recent call last):
  File "/opt/app-root/lib/python3.6/site-packages/tornado/web.py", line 1704, in _execute
    result = await result
  File "/opt/app-root/src/cara/apps/calculator/__init__.py", line 114, in post
    report_generator.build_report, base_url, form,
  File "/opt/rh/rh-python36/root/usr/lib64/python3.6/concurrent/futures/process.py", line 452, in submit
    raise BrokenProcessPool('A child process terminated '
concurrent.futures.process.BrokenProcessPool: A child process terminated abruptly, the process pool is not usable anymore

It is possible that this is because of nested ProcessPoolExecutors, but haven't been able to reproduce locally. For now, I will roll it back as this is impacting users.

Edited by Philip Elson

Merge request reports