-
Notifications
You must be signed in to change notification settings - Fork 910
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
Use RpcRequest in PlanExecutor configs #3405
Conversation
|
f59c8c5
to
7c00911
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
7c00911
to
aa2947b
Compare
Because there has been no activity on this PR for 14 days since it was merged, it has been automatically locked. Please open a new issue if it requires a follow up. |
This PR replaces the
notificationName
andparams
arguments of theRpcSubscriptionsPlanExecutor
function by a singlerequest
argument of typeRpcRequest
. This request argument is meant to represent the user intent to request a subscription from the RPC Subscriptions packages.Note: A changeset for this PR is included in #3407.