Skip to content

Handle compaction events in latest-at cache subscriber#7181

Merged
Wumpf merged 3 commits intomainfrom
cmc/compacted_latestat_invalidation
Aug 14, 2024
Merged

Handle compaction events in latest-at cache subscriber#7181
Wumpf merged 3 commits intomainfrom
cmc/compacted_latestat_invalidation

Conversation

@teh-cmc
Copy link
Copy Markdown
Contributor

@teh-cmc teh-cmc commented Aug 14, 2024

This is the exact same thing as #7161, except it's for latest-at (and requires a bit more care, because latest-at is weird).

Checklist

  • I have read and agree to Contributor Guide and the Code of Conduct
  • I've included a screenshot or gif (if applicable)
  • I have tested the web demo (if applicable):
  • The PR title and labels are set such as to maximize their usefulness for the next release's CHANGELOG
  • If applicable, add a new check to the release checklist!
  • If have noted any breaking changes to the log API in CHANGELOG.md and the migration guide

To run all checks from main, comment on the PR with @rerun-bot full-check.

@teh-cmc teh-cmc added 🪳 bug Something isn't working 📉 performance Optimization, memory use, etc exclude from changelog PRs with this won't show up in CHANGELOG.md labels Aug 14, 2024
@teh-cmc teh-cmc force-pushed the cmc/compacted_latestat_invalidation branch from 97407fc to eefd993 Compare August 14, 2024 11:56
@Wumpf Wumpf self-requested a review August 14, 2024 12:06
Copy link
Copy Markdown
Member

@Wumpf Wumpf left a comment

Choose a reason for hiding this comment

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

looks very familiar indeed

@Wumpf Wumpf merged commit 5aa4ba0 into main Aug 14, 2024
@Wumpf Wumpf deleted the cmc/compacted_latestat_invalidation branch August 14, 2024 12:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🪳 bug Something isn't working exclude from changelog PRs with this won't show up in CHANGELOG.md 📉 performance Optimization, memory use, etc

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants