temporalio / sdk-python

Temporal Python SDK
MIT License
473 stars 77 forks source link

Tweak error message on workflow cancellation #583

Closed dandavison closed 4 months ago

dandavison commented 4 months ago

Workflow cancellation results in WorkflowFailureError("workflow execution failed"). It would make sense for that to be WorkflowFailureError("workflow execution cancelled").

dandavison commented 4 months ago

Actually, this suggestion isn't appropriate: it makes sense that the outer message has a natural language string matching its class name; the cancellation exception is contained within the outer exception as the __cause__ (exposed as .cause)