Skip to content

fix(iterator): always initialize pipeline output template #997

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Apr 10, 2025

Conversation

jvallesm
Copy link
Collaborator

@jvallesm jvallesm commented Apr 10, 2025

Because

  • Streamed executions with iterators failed because the _output key
    wasn't set in the iterator's child workflow memory. This is
    initialized on InitComponentsActivity, which is run only on parent
    workflows.

This commit

  • Intializes this field on workflow memory creation. ComponentActivity
    will use it to update the (child) pipeline output at the end of the
    sub-component execution.

Because

- Streameed executions with iterators failed because the `_output` key
  wasn't set in the iterator's child workflow memory. This is
  initialized on `InitComponentsActivity`, which is run only on parent
  workflows.

This commit

- Intializes this field on workflow memory creation. `ComponentActivity`
  will use it to update the (child) pipeline output at the end of the
  sub-component execution.
Copy link
Member

@donch1989 donch1989 left a comment

Choose a reason for hiding this comment

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

Great finding!

@jvallesm jvallesm merged commit 3020465 into main Apr 10, 2025
13 checks passed
@jvallesm jvallesm deleted the jvalles/fix-iterator-path branch April 10, 2025 19:59
jvallesm pushed a commit that referenced this pull request Apr 16, 2025
🤖 I have created a release *beep* *boop*
---


##
[0.52.2-beta](v0.52.1-beta...v0.52.2-beta)
(2025-04-10)


### Bug Fixes

* **iterator:** always initialize pipeline output template
([#997](#997))
([3020465](3020465))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Status: No status
Development

Successfully merging this pull request may close these issues.

3 participants