Fail the run if publish thread pool times out

[bentsherman]

Close #5577

[netlify]

✅ Deploy Preview for nextflow-docs-staging canceled.

Name	Link
🔨 Latest commit	b4aa73d
🔍 Latest deploy log	https://app.netlify.com/sites/nextflow-docs-staging/deploys/675c5feb7f12fa000845190a

[bentsherman]

@pditommaso ready for review. It will fail the run if workflow.output.ignoreErrors is false and the finalizer or publish pool times out

[pditommaso]

Would have not been simpler to throw an exception here and rely on the abort mechanism?

[bentsherman]

Probably would be simpler. In that case the abort() would be called in the middle of destroy(), is that expected to work?

[pditommaso]

I think the abort will be called implicitly by throwing an exception. see here

nextflow/modules/nextflow/src/main/groovy/nextflow/script/ScriptRunner.groovy

Line 150 in 237894a

session.abort(e)

[bentsherman]

Right, just wondering if abort() has some implicit assumption that destroy() has not been called yet. But from the code it looks safe. Simplified things a bit