Skip to content

Asynchronous overhaul #46

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.

Sign up for GitHub

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

Jump to bottom
Merged
NotJustAnna merged 17 commits into from
Apr 21, 2020
Merged

Asynchronous overhaul #46

NotJustAnna merged 17 commits into from
Apr 21, 2020

Conversation

@NotJustAnna
Copy link
Member

@NotJustAnna NotJustAnna commented Apr 20, 2020 •
edited
Loading

This pull request implements the following:

  • Support for daemon threads on the response pump.
  • Detaching response parsing from the read loop.
  • Asynchronous connect/reconnect/close methods.
  • ReqlAst queries are more resource efficient by replacing empty OptArgs with null.
  • A lot of documentation.

Reverted:

  • Connection#unwrapLists was removed.
  • db-url support for auth keys was removed.

Deprecated:

  • Deprecated auth keys support.

Adrian Todt and others added 8 commits March 22, 2020 16:12
@NotJustAnna NotJustAnna added this to the v2.4.3 milestone Apr 20, 2020
@NotJustAnna NotJustAnna mentioned this pull request Apr 20, 2020
Closed
@NotJustAnna NotJustAnna force-pushed the feature/async-connection branch from b60f939 to 3181f5a Compare April 20, 2020 03:37
Adrian Todt added 9 commits April 20, 2020 00:53
Merge branch 'master' into develop
456f186
Asynchronous variances for connect, reconnect and close methods.
2cfa153 
Refactored Connection Builder.
Adds persistent/daemon thread support.
Support for asynchronous socket creation.
Deprecated some methods.
Optimized queries.
19b4afd
Response pump is now faster due to Response parsing being done in ano…
5df36c7 
…ther thread instead on the read-loop.
Due to the previous commit, this doesn't need to be in a different th…
b0474b1 
…read anymore.
Revert "unwrapLists now can be configured at runtime"
0379682 
This reverts commit 5368e15
Merge remote-tracking branch 'origin/feature/async-connection' into f…
0e8b499 
…eature/async-connection
Deprecated support for auth-key, documented all methods.
3f50bdf
Fixed tests, non-breaking support for async connection factories.
6fc82f5
@NotJustAnna NotJustAnna merged commit 8fc00d5 into develop Apr 21, 2020
@NotJustAnna NotJustAnna deleted the feature/async-connection branch April 21, 2020 16:53
@NotJustAnna NotJustAnna mentioned this pull request May 8, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

v2.4.3

Development

Successfully merging this pull request may close these issues.

2 participants

@NotJustAnna