Enable Async turbopuffer API #60
Open
Conversation
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
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.
Enables Python asyncio turbopuffer calls. This implements all the tubobuffer entry points to have an async counterpart as well as keeping the existing synchronous API. The synchronous API is implemented to use the async API which eliminates code duplication, this also means the unit tests provide the same coverage for the async API.
Some tests which depended on implementation details needed some surgery to work with the new async implementation (not fun since mocking the async impl required involved mocks).
Added some docs to README.
This is an extensive change and it re-implements virtually all code paths. Very extensive testing should be performed before releasing to the wild.