Support dispatch session state #106
Open
Conversation
This file contains 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
|
|
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
Hiiii, @weishaolun welcome!馃帄 Thanks for taking the effort to make our project better! 馃檶 Keep making such awesome contributions!
|
Hi @weishaolun thanks for your contribution! BTW, welcome to amend your commit message following our commit message conventions, eg: the commit title should be under 50 words max. |
weishaolun
force-pushed
the
support_dispatch_session_state
branch
2 times, most recently
from
1c9ce9c
to
21e8af0
Compare
yjhjstz
reviewed
Aug 31, 2023
yjhjstz
reviewed
Aug 31, 2023
yjhjstz
reviewed
Aug 31, 2023
weishaolun
force-pushed
the
support_dispatch_session_state
branch
from
21e8af0
to
808446d
Compare
yjhjstz
approved these changes
Nov 7, 2023
|
Hi @weishaolun Thanks for your contribution. Please use "rebase" instead of "merge" for your commits to stay up to date with the main branch. Could you clear your commit history before your PR is merged? |
weishaolun
force-pushed
the
support_dispatch_session_state
branch
from
d7446d4
to
808446d
Compare
Extending the query dispatch mechanism, allowing extra data to be dispatched. To dispatch extra data, you just need to register an extra dispatch with a unique name and a pair of functions for generating and parsing the data to be dispatched.
weishaolun
force-pushed
the
support_dispatch_session_state
branch
from
808446d
to
39e239d
Compare
weishaolun
force-pushed
the
support_dispatch_session_state
branch
from
39e239d
to
d0960ba
Compare
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.
Change logs
Why are the changes needed?
To allow dispatch session state and set up the session state in a extension.
Does this PR introduce any user-facing change?
If yes, please clarify the previous behavior and the change this PR proposes.
How was this patch tested?
Please detail how the changes were tested, including manual tests and any relevant unit or integration tests.
Contributor's Checklist
Here are some reminders and checklists before/when submitting your pull request, please check them:
make installcheckmake -C src/test installcheck-cbdb-parallel