Skip to content
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

wasm: reset call context on failure #17695

Merged
merged 2 commits into from
Apr 7, 2024

Conversation

rockwotj
Copy link
Contributor

@rockwotj rockwotj commented Apr 6, 2024

In the case of an error we still need to reset the call context or things could break when restarting the module.

Backports Required

  • none - not a bug fix
  • none - this is a backport
  • none - issue does not exist in previous branches
  • none - papercut/not impactful enough to backport
  • v23.3.x
  • v23.2.x

Release Notes

  • none

In the case of an error we still need to reset the call context.

Signed-off-by: Tyler Rockwood <rockwood@redpanda.com>
Signed-off-by: Tyler Rockwood <rockwood@redpanda.com>
@github-actions github-actions bot added area/redpanda area/wasm WASM Data Transforms labels Apr 6, 2024
@rockwotj rockwotj requested review from dotnwat and oleiman April 6, 2024 15:15
@rockwotj rockwotj self-assigned this Apr 6, 2024
@piyushredpanda piyushredpanda added this to the v24.1-rc1 milestone Apr 6, 2024
@piyushredpanda piyushredpanda merged commit d4dc91b into redpanda-data:dev Apr 7, 2024
18 checks passed
@rockwotj rockwotj deleted the reset-call-ctx branch April 8, 2024 01:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/redpanda area/wasm WASM Data Transforms
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants