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.

Sign up for GitHub

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

Jump to bottom

ingest: Prevent multi-ledger compaction in ChangeCompactor #5607

Open
urvisavla opened this issue Feb 18, 2025 · 0 comments
Open

ingest: Prevent multi-ledger compaction in ChangeCompactor #5607

urvisavla opened this issue Feb 18, 2025 · 0 comments
Labels
cdp-horizon-scrum good first issue

Comments

@urvisavla
Copy link
Contributor

What

Add a check to prevent a single ChangeCompactor instance from processing changes from multiple ledger sequences.

See #5587 (comment) for more details.

Why

The ChangeCompactor is designed to combine changes for the same ledger entry within a single ledger. Currently, it's possible to use a single ChangeCompactor instance to process changes from multiple ledgers.

What Alternatives are there?

Leave it to the users of the ingest library to ensure it is used correctly for a single ledger.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
cdp-horizon-scrum good first issue
Projects
Platform Scrum
Status: To Do
Milestone
No milestone
Development

No branches or pull requests
1 participant
@urvisavla