Skip to content
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

Prepare multiplexing support for database plugins #16995

Merged
merged 3 commits into from
Sep 6, 2022

Conversation

maxcoulombe
Copy link
Contributor

@maxcoulombe maxcoulombe commented Sep 2, 2022

Preparing multiplexing support for all built-in database plugins. Multiplexing is only supported for external plugins so this change is effectively a noop but prepares the field for future projects. As it is a noop for the time being I decided against including a changelog but let me know it you think it'd make sense to include one.

All plugins were tested locally to confirm they still work as expected but share a single process between all plugin instances.

Copy link
Contributor

@vinay-gopalan vinay-gopalan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good! Think we might have missed one

@vinay-gopalan
Copy link
Contributor

Are we sure that this PR wouldn't warrant a changelog? I think we might have to add one as an improvement?

Copy link
Contributor

@vinay-gopalan vinay-gopalan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks for taking care of these! :)

@maxcoulombe maxcoulombe merged commit 59114c5 into main Sep 6, 2022
@maxcoulombe maxcoulombe deleted the vault-3725-databaseMultiplexingMigration branch September 6, 2022 18:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants