[Question]: About what happens when panic occurred inside a worker

[vvakame]

Questions with details

Hi! My team uses ants a lot. Thanks for a great product!

Document of Options.PanicHandler said below ¹

if nil, panics will be thrown out again from worker goroutines.

In the actual implementation, if PanicHandler is nil, it appears to be suppressed after logging output. ²

If I want to detect a panic (or propagate panic to main process) after Invoke called, should I use WithPanicHandler with own implementations?

Code snippets (optional)

Footnotes

1. https://github.com/panjf2000/ants/blob/a44594205e16818434d27f02da3f81c26e51a74a/options.go#L57-L58 ↩

2. https://github.com/panjf2000/ants/blob/a44594205e16818434d27f02da3f81c26e51a74a/worker_func.go#L62 ↩

[panjf2000]

WithPanicHandler was first introduced by #23 and the behavior matched the docs, but the behavior had been changed since this commit and I forgot to update the docs. Thanks for pointing it out!

If I want to detect a panic (or propagate panic to main process) after Invoke called, should I use WithPanicHandler with own implementations?

As for your use case, yes, you can call panic in your PanicHandler.

[vvakame]

thanks to explanations!