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

[Feature] Support for Lightweight Ping Syntax #214

Closed
jchrys opened this issue Jan 22, 2024 · 0 comments · Fixed by #217
Closed

[Feature] Support for Lightweight Ping Syntax #214

jchrys opened this issue Jan 22, 2024 · 0 comments · Fixed by #217
Assignees
@mirromutth
Labels
enhancement New feature or request
Milestone
1.1.0

Comments

@jchrys
Copy link
Collaborator

jchrys commented Jan 22, 2024

Is your feature request related to a problem? Please describe.
R2DBC MySQL lacks the efficient /* ping */ syntax for connection checks, causing unnecessary overhead.

Describe the solution you'd like
Introduce the /* ping */ syntax in R2DBC MySQL, mirroring MySQL Connector/J.

Additional context
ref: https://dev.mysql.com/doc/connector-j/en/connector-j-usagenotes-j2ee-concepts-connection-pooling.html
@jchrys jchrys added the enhancement New feature or request label Jan 22, 2024
@mirromutth mirromutth self-assigned this Jan 25, 2024
@mirromutth mirromutth added this to the 1.1.0/0.10.0 milestone Jan 25, 2024
@mirromutth mirromutth mentioned this issue Jan 25, 2024
Merged
@mirromutth mirromutth linked a pull request Jan 25, 2024 that will close this issue
Add lightweight ping syntax support #217
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mirromutth mirromutth

Labels
enhancement New feature or request
Projects
None yet
Milestone
1.1.0
Development

Successfully merging a pull request may close this issue.

Add lightweight ping syntax support asyncer-io/r2dbc-mysql
2 participants
@mirromutth @jchrys