-
Notifications
You must be signed in to change notification settings - Fork 669
Min 4k TKV log page size; revise device sector size setting; DefaultInitialIORecordSize overridable by operation, session, or store #1853
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
Merged
Merged
Changes from all commits
Commits
Show all changes
24 commits
Select commit
Hold shift + click to select a range
4f4fbfe
Keep AsyncGetFromDisk calling size to be under SectorSize unless spec…
TedHartMS e9ef3b0
Tweak for GetAndPopulateReadBuffer read length
TedHartMS 2b5b635
Safekeeping checkin, mostly test conversion to support 4k sectors whi…
TedHartMS 161c95a
Finish conversion of tests to use 4k pages, which will work with any …
TedHartMS 80b521a
Fixes so either 512b or 4kb sector sizes pass the tests
TedHartMS 76b1c0d
Merge remote-tracking branch 'origin/main' into tedhar/io-4k
TedHartMS b95908a
Fix perf regression for new multilevel InitialIOSize spec
TedHartMS d034f35
Return GetDeviceSectorSize to use GetDiskFreeSpace and return MinDevi…
TedHartMS 21c48e2
Merge 'main'
TedHartMS 36f34cc
format
TedHartMS edf6b80
Fix comment for default for Garnet InitialIORecordSize option
TedHartMS fe33014
Address Copilot comments; restore previous version of GetDeviceSector…
TedHartMS 3494532
Make WaitForFrameLoad rethrow OperationCanceledException
TedHartMS 51bbca1
temporarily disable test that needs updating
TedHartMS ecf7902
Fix ObjectIterationPushLockTest
TedHartMS ce2cbfa
Merge remote-tracking branch 'origin/main' into tedhar/io-4k
TedHartMS b97d421
Fix ObjectIterationPushLockTests
TedHartMS a7f77b4
Merge branch 'main' into tedhar/io-4k
TedHartMS e9a6bd6
Fix MultiDatabaseSaveInProgressTest
TedHartMS 8eae606
Merge branch 'tedhar/io-4k' of https://github.com/microsoft/Garnet in…
TedHartMS 260fab0
merge 'main'
TedHartMS 88b3510
Add WaitUntilNextSecond() to handle LASTSAVE returning only second-le…
TedHartMS 9147319
format
TedHartMS 14e6aeb
merge 'main'
TedHartMS File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.