-
Notifications
You must be signed in to change notification settings - Fork 420
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
Basic client RPC executor #1693
Merged
Merged
Commits on Oct 30, 2023
-
Motivation: Clients can execute RPCs in a few different ways: straight up as a single attempt, with retries, or with hedging. This is done by the `ClientRPCExecutor`, the bones of which are added in this PR. At a high level, the executor takes a transport, request, serializer, deserializer and response handler and executes the request against a transport and executes the response handler if a response is received or synthesized locally. The executor only returns once the response has been handled. Modifications: - Add the `ClientRPCExecutor` and the one-shot implementation (hedging and retries will follow later) - The `ClientRPCExecutor` uses a `ClientStreamExecutor` which deals in serialized streams - Add a testing harness, which includes type erased transports (and a basic in process transport for testing) and different server behaviours. Result: Can execute one-shot requests.
Configuration menu - View commit details
-
Copy full SHA for bbfb993 - Browse repository at this point
Copy the full SHA bbfb993View commit details -
Configuration menu - View commit details
-
Copy full SHA for 463ed93 - Browse repository at this point
Copy the full SHA 463ed93View commit details -
Configuration menu - View commit details
-
Copy full SHA for c7deb7d - Browse repository at this point
Copy the full SHA c7deb7dView commit details -
Configuration menu - View commit details
-
Copy full SHA for c772637 - Browse repository at this point
Copy the full SHA c772637View commit details
Commits on Oct 31, 2023
-
Configuration menu - View commit details
-
Copy full SHA for f68d023 - Browse repository at this point
Copy the full SHA f68d023View commit details -
Configuration menu - View commit details
-
Copy full SHA for d42978c - Browse repository at this point
Copy the full SHA d42978cView commit details -
Configuration menu - View commit details
-
Copy full SHA for f8d8ebd - Browse repository at this point
Copy the full SHA f8d8ebdView commit details -
Configuration menu - View commit details
-
Copy full SHA for 834f105 - Browse repository at this point
Copy the full SHA 834f105View commit details -
Configuration menu - View commit details
-
Copy full SHA for 431da59 - Browse repository at this point
Copy the full SHA 431da59View commit details -
Configuration menu - View commit details
-
Copy full SHA for 924f004 - Browse repository at this point
Copy the full SHA 924f004View commit details -
Configuration menu - View commit details
-
Copy full SHA for 5ee3de3 - Browse repository at this point
Copy the full SHA 5ee3de3View commit details
Commits on Nov 1, 2023
-
Configuration menu - View commit details
-
Copy full SHA for 8545e25 - Browse repository at this point
Copy the full SHA 8545e25View commit details -
Configuration menu - View commit details
-
Copy full SHA for 113f379 - Browse repository at this point
Copy the full SHA 113f379View commit details
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.