-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
vendor: bump pebble to af0d71572c7bd24deb066d09eee646b6cb7460a4 #49285
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed 2 of 2 files at r1.
Reviewable status: complete! 0 of 0 LGTMs obtained (waiting on @itsbilal)
* internal/private: ratchet visible sequence number too * internal/rate: add Limiter.Delay{,N} * db: refactor tableCache.estimateDiskUsage into withReader * internal/cache: optimize common case of Cache.Delete() * internal/cache: remove weak cache handles * internal/randvar: optimize Uniform.IncMax * internal/{manual,rawalloc}: remove empty assembly files * internal/replay: assign ingested tables correct sequence number * db: wait for pending writes during ingestion * db: replace tableCacheShard LRU with CLOCK-Pro * internal/randvar: reduce exclusive locks in randvar * cmd/pebble: miscellaneous ycsb benchmark improvements * internal/rate: change Reserve{,N} to return a value Fixes cockroachdb#49154 Release note: None
04fa19b
to
389b94f
Compare
TFTR! bors r+ |
Build failed |
SQL logictest failures that look unrelated to this PR. Retrying. bors r+ |
Merge conflict (retrying...) |
(you have won the race, i can wait) |
Build failed (retrying...) |
bors r+ |
Already running a review |
o_o bors r+ edit: nvm can't read |
Already running a review |
Build succeeded |
Fixes #49154
Release note: None