[new release] index (1.3.1) #18607
Merged
[new release] index (1.3.1) #18607
Conversation
This file contains 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
samoht
commented
Apr 29, 2021
samoht
commented
Apr 29, 2021
samoht
force-pushed
the
release-index-1.3.1
branch
from
7a594d1
to
a323a1e
Compare
samoht
changed the title
CHANGES: - Reduce allocations during merge (mirage/index#274, mirage/index#277) - Protect concurrent syncs with a lock (mirage/index#309) - Fixed a performance issue for `Index.sync` when there is a blocking merge in progress: the `log_async` file was not cached properly and fully reloaded from disk every time. (mirage/index#310) - Release the merge lock if a merge raises an exception (mirage/index#312) - Added fsync after `Index.clear` to signal more quickly to read-only instances than something has changed in the file (mirage/index#308) - Specialise `IO.v` to create read-only or read-write instances. (mirage/index#291) - `clear` removes the files on disks and opens new ones containing only the header. (mirage/index#288, mirage/index#307, mirage/index#317)
samoht
force-pushed
the
release-index-1.3.1
branch
from
a323a1e
to
16ff00f
Compare
mseri
reviewed
Apr 29, 2021
Co-authored-by: Marcello Seri <mseri@users.noreply.github.com>
Co-authored-by: Marcello Seri <mseri@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
A platform-agnostic multi-level index for OCaml
CHANGES:
Fixed
Reduce allocations during merge (Reduce allocations during merge mirage/index#274, merge: only decode the value when necessary mirage/index#277)
Protect concurrent syncs with a lock (protect sync with a lock mirage/index#309)
Fixed a performance issue for
Index.syncwhen there is a blocking merge inprogress: the
log_asyncfile was not cached properly and fully reloadedfrom disk every time. (Do not reload log_async on every sync mirage/index#310)
Release the merge lock if a merge raises an exception (index: release the merge lock if a merge raises an exception mirage/index#312)
Added fsync after
Index.clearto signal more quickly to read-only instancesthan something has changed in the file (use fsync to flush metadata when reoperning a file with new headers mirage/index#308)
Changed
Specialise
IO.vto create read-only or read-write instances. (Specialisevto readonly and readwrite instances mirage/index#291)clearremoves the files on disks and opens new ones containing only theheader. (Remove files on disk on
clearmirage/index#288, Simplifications on top of #288 mirage/index#307, Simplify clear again mirage/index#317)