Skip to content

TaskOrchestrationContext.allOf should throw TaskFailureException instead of RuntimeException #54

@cgillum

Description

@cgillum

The implementation of this method in TaskOrchestrationExecutor.java currently throws a RuntimeException if any of the inner tasks fails. This seems wrong because tasks are only supposed to fail with TaskFailedException.

One challenge is that TaskFailedException is designed to represent a single named task. We may need to create a new exception type for composite exceptions. Alternatively, we could surface the details of the first exception. It may be useful to use the current .NET implementation design as a reference for this design.

As part of this work item, we should also do the following:

  • Update the API documentation for TaskOrchestrationContext.allOf to accurately reflect the expected exception behavior
  • Write a unit or integration test that ensures the correct behavior

Metadata

Metadata

Type

No type

Projects

No projects

Relationships

None yet

Development

No branches or pull requests

Issue actions