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.

Sign up for GitHub

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

Jump to bottom

[enhancement] add a session manage module #180

Open
xuanyuan300 opened this issue Jul 8, 2022 · 3 comments
Open

[enhancement] add a session manage module #180

xuanyuan300 opened this issue Jul 8, 2022 · 3 comments
Assignees
@xuanyuan300
Labels
app:pisa-proxy Pisa-Proxy related changes type:enhancement New feature or request
Milestone
v0.3.0

Comments

@xuanyuan300
Copy link
Contributor

Feature Request

Is your feature request related to a problem? Please describe:

Describe the feature you'd like:

After added the readwritesplitting feature, the processing of ConnAttr has been added. Currently, the attrs processed are charset, autocommit, but the processing logic is integrated in Pool, and Pool is a public component, different databases may also there are different session attrs, so ConnAttr should be separated from Pool.

Describe alternatives you've considered:

Add a specifiy session manage module.

Teachability, Documentation, Adoption, Migration Strategy:
@xuanyuan300 xuanyuan300 added type:enhancement New feature or request app:pisa-proxy Pisa-Proxy related changes labels Jul 8, 2022
@xuanyuan300 xuanyuan300 added this to the v0.2.0 milestone Jul 8, 2022
@xuanyuan300 xuanyuan300 self-assigned this Jul 8, 2022
@mlycore mlycore modified the milestones: v0.2.0, v0.3.0 Aug 4, 2022
@xuanyuan300
Copy link
Contributor Author

WIP, this issue will reslove in v0.3.0.

@mlycore
Copy link
Member

mlycore commented Aug 14, 2022

It is useful for backend database endpoint rolling upgrade without any discruptions of applications.

@xuanyuan300
Copy link
Contributor Author

In pr #274, Added Session trait and SessionMut trait.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@xuanyuan300 xuanyuan300

Labels
app:pisa-proxy Pisa-Proxy related changes type:enhancement New feature or request
Projects
None yet
Milestone
v0.3.0
Development

No branches or pull requests
2 participants
@xuanyuan300 @mlycore