Skip to content

feat: update OrchestrationDefinition API model#50

Merged
paullatzelsperger merged 8 commits into
eclipse-cfm:mainfrom
paullatzelsperger:feat/adapt_orchdef_api_model
Feb 23, 2026
Merged

feat: update OrchestrationDefinition API model#50
paullatzelsperger merged 8 commits into
eclipse-cfm:mainfrom
paullatzelsperger:feat/adapt_orchdef_api_model

Conversation

@paullatzelsperger
Copy link
Copy Markdown
Contributor

@paullatzelsperger paullatzelsperger commented Feb 19, 2026

This PR changes the data model of the orchestration definition API by:

  • introducing the OrchestrationTemplate: OrchestrationDefinition objects are generated based off of that template and linked to it with the templateRef property
  • added a get-by-templateref endpoint which allows to look for all OrchestrationDefinitions that were generated on a certain template.

@paullatzelsperger paullatzelsperger force-pushed the feat/adapt_orchdef_api_model branch from 849d844 to 15f8452 Compare February 23, 2026 07:24
@paullatzelsperger paullatzelsperger marked this pull request as ready for review February 23, 2026 07:26
Copy link
Copy Markdown
Contributor

@jimmarino jimmarino left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we get rid of the Stream abstraction and use Seq/Seq2 instead? The Stream abstraction isn't needed, and any required filtering/transformation can be done using composable Seq/Seq2 in a range using functions:

for n := range MapBar(FilterFoo(getResults())) {
    // Do something
}

The approach is used in the memory and SQL stores. Any general functions (if they are needed) can go into common/collections but I'm not sure if such functions are neded.

@paullatzelsperger paullatzelsperger force-pushed the feat/adapt_orchdef_api_model branch from 24505b7 to 4fbb0f7 Compare February 23, 2026 09:57
@paullatzelsperger paullatzelsperger merged commit 9f09efe into eclipse-cfm:main Feb 23, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants