Skip to content

Subscription: add removeUserData for messages#17460

Open
VGalaxies wants to merge 4 commits intoapache:masterfrom
VGalaxies:codex/subscription-remove-user-data
Open

Subscription: add removeUserData for messages#17460
VGalaxies wants to merge 4 commits intoapache:masterfrom
VGalaxies:codex/subscription-remove-user-data

Conversation

@VGalaxies
Copy link
Copy Markdown
Contributor

@VGalaxies VGalaxies commented Apr 12, 2026

Summary

  • add SubscriptionMessage.removeUserData() for record messages while preserving commit context
  • release record handler payload state after consume and keep tsfile handling unchanged
  • store only SubscriptionCommitContext in pull consumer auto-commit state
  • add subscription integration tests under integration-test for removeUserData and auto-commit behavior

This PR was primarily authored with Codex using gpt-5.4 xhigh and then hand-reviewed by me. I AM responsible for every change made in this PR. I aimed to keep it aligned with our goals, though I may have missed minor issues. Please flag anything that feels off, I'll fix it quickly.

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