search

celery / billiard

Multiprocessing Pool Extensions
Other
419 stars 252 forks source link

ExceptionInfo.exception should be an exception #372

Closed woutdenolf closed 2 years ago

woutdenolf commented 2 years ago

Closes #371

Celery expects ExceptionInfo.exception to be an exception:

https://github.com/celery/celery/blob/feaad3f9fdf98d0453a07a68e307e48c6c3c2550/celery/app/trace.py#L239

This MR makes sure it is an exception while at the same time preserving the cause while pickling (tested with Celery 5.3.0b1).

woutdenolf commented 2 years ago

FYI I don't have permissions to merge.

auvipy commented 2 years ago

shit I missed it

auvipy commented 2 years ago

need to release a bug fix tomorrow