Skip to content

feat: [Parallel] Add parallel result#246

Merged
zhongkechen merged 2 commits intomainfrom
prallel/result
Mar 21, 2026
Merged

feat: [Parallel] Add parallel result#246
zhongkechen merged 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

@zhongkechen zhongkechen merged commit a19fc08 into main Mar 21, 2026
12 checks passed
@zhongkechen zhongkechen deleted the prallel/result branch March 21, 2026 16:03
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