Skip to content

feat: [Parallel] Add parallel result#246

Open
wangyb-A wants to merge 2 commits intomainfrom
prallel/result
Open

feat: [Parallel] Add parallel result#246
wangyb-A wants to merge 2 commits intomainfrom
prallel/result

Conversation

@wangyb-A
Copy link
Contributor

@wangyb-A wangyb-A commented Mar 20, 2026

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Issue Link, if available

Description

  • Removed the generic type parameter — ParallelOperation is now a concrete class returning ParallelResult
  • get() now returns a ParallelResult (total, succeeded, failed branch counts + completion status) instead of null

Demo/Screenshots

Checklist

  • I have filled out every section of the PR template
  • I have thoroughly tested this change

Testing

iShot_2026-03-20_16 54 58

Unit Tests

Have unit tests been written for these changes? Yes

Integration Tests

Have integration tests been written for these changes? Not yet

Examples

Has a new example been added for the change? (if applicable) Existing

@wangyb-A wangyb-A marked this pull request as ready for review March 20, 2026 23:57
@wangyb-A wangyb-A requested a review from a team March 20, 2026 23:57
@wangyb-A
Copy link
Contributor Author

Next steps:

  • Reuse completion config
  • Integration tests
  • Reevaluate early termination behaviour

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.

1 participant