Skip to content

[lmdb] Update LMDB to latest upstream #172

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 2 commits into from
Dec 7, 2023
Merged

Conversation

bnbarham
Copy link
Contributor

Summary of interesting changes:

  • Windows compilation fixes
  • Fix for errant MDB_PAGE_FULL on mdb_put which could happen when records had non-uniform sizes (this is the upstream patch for the fix in 6417ff8)
  • Alignment fix for SPARC (probably doesn't impact us)
  • Fix for replacing values with MDB_DUPSORT leading to a broken database

Summary of interesting changes:
  * Windows compilation fixes
  * Fix for errant MDB_PAGE_FULL on mdb_put which could happen when
    records had non-uniform sizes (this is the upstream patch for
    the fix in 6417ff8)
  * Alignment fix for SPARC (probably doesn't impact us)
  * Fix for replacing values with MDB_DUPSORT leading to a broken
    database
We had already fixed `commit`, fix `abort` as well.
@bnbarham
Copy link
Contributor Author

@swift-ci please test

@bnbarham
Copy link
Contributor Author

@swift-ci please test Windows platform

1 similar comment
@bnbarham
Copy link
Contributor Author

@swift-ci please test Windows platform

@bnbarham
Copy link
Contributor Author

@swift-ci please clean test Windows platform

@bnbarham
Copy link
Contributor Author

@swift-ci please test Windows platform

@bnbarham
Copy link
Contributor Author

bnbarham commented Dec 1, 2023

@swift-ci please test

1 similar comment
@bnbarham
Copy link
Contributor Author

bnbarham commented Dec 7, 2023

@swift-ci please test

@bnbarham bnbarham merged commit 5eac09c into swiftlang:main Dec 7, 2023
@bnbarham bnbarham deleted the update-lmdb branch December 7, 2023 17:16
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