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

release-25.1: go.mod: bump Pebble to 7c863c9c3bd8 #139267

Merged
merged 1 commit into from
Jan 16, 2025

Conversation

jbowens
Copy link
Collaborator

@jbowens jbowens commented Jan 16, 2025

Changes:

  • 7c863c9c sstable: copy filter block during suffix rewriting
  • 94fd2294 sstable: rewrite multiple times in TestRewriteSuffixProps
  • 16da0a9d objstorage: sometimes mangle MemObj.Write input in invariants builds
  • 4d8664cd sstable: revert change to binary search calculation

Release note: none.
Epic: none.
Release justification: fix AddSSTable corruption issue

Changes:

 * [`7c863c9c`](cockroachdb/pebble@7c863c9c) sstable: copy filter block during suffix rewriting
 * [`94fd2294`](cockroachdb/pebble@94fd2294) sstable: rewrite multiple times in TestRewriteSuffixProps
 * [`16da0a9d`](cockroachdb/pebble@16da0a9d) objstorage: sometimes mangle MemObj.Write input in invariants builds
 * [`4d8664cd`](cockroachdb/pebble@4d8664cd) sstable: revert change to binary search calculation

Release note: none.
Epic: none.
@jbowens jbowens requested review from a team and RaduBerinde January 16, 2025 21:34
@jbowens jbowens requested a review from a team as a code owner January 16, 2025 21:34
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@jbowens jbowens changed the title crl-release-25.1: go.mod: bump Pebble to 7c863c9c3bd8 release-25.1: go.mod: bump Pebble to 7c863c9c3bd8 Jan 16, 2025
@jbowens
Copy link
Collaborator Author

jbowens commented Jan 16, 2025

TFTR!

@jbowens jbowens merged commit a3c6dfe into cockroachdb:release-25.1 Jan 16, 2025
20 checks passed
@jbowens jbowens deleted the bump-pebble-7c863c9c3bd8 branch January 16, 2025 23:29
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.

3 participants