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

Allow for concurrent connections with aliases #196

Merged
merged 15 commits into from
Nov 18, 2023
Merged

Conversation

bhirsz
Copy link
Member

@bhirsz bhirsz commented Nov 9, 2023

This PR was based on #149 hovewer there were major changes to design.

Closes #102

@bhirsz bhirsz requested a review from amochin November 9, 2023 18:06
@bhirsz bhirsz force-pushed the aliased_connection branch 2 times, most recently from b0d1b25 to c4f2a72 Compare November 10, 2023 10:27
@amochin
Copy link
Collaborator

amochin commented Nov 18, 2023

I've made some improvements in docs and tests, but everything looks brilliant, so I'm merging it.
Thank you so much for the great job, @bhirsz!

@amochin amochin merged commit f368032 into master Nov 18, 2023
10 checks passed
@amochin amochin deleted the aliased_connection branch November 18, 2023 20:31
@nczita
Copy link
Contributor

nczita commented Nov 22, 2023

Thanks, @bhirsz and @amochin for taking this on!
I was too occupied lately to deal with it, but I like your improvements very much.

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.

support connect 2 or more database
3 participants