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
Named query for Conn #780
base: master
Are you sure you want to change the base?
Named query for Conn #780
Conversation
Hello, @ardan-bkennedy, and I recently stepped in to help maintain this project. |
Hello.
Yes, I still need it.
From: Florin Pățan ***@***.***>
Sent: Thursday, February 1, 2024 4:47 PM
To: jmoiron/sqlx ***@***.***>
Cc: Козлов Евгений ***@***.***>; Author ***@***.***>
Subject: Re: [jmoiron/sqlx] Named query for Conn (PR #780)
Hello, @ardan-bkennedy<https://github.com/ardan-bkennedy>, and I recently stepped in to help maintain this project.
We are sorting the opened issues and pull requests and would like to know if you still NEED this merged.
Thank you for your contribution.
—
Reply to this email directly, view it on GitHub<#780 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AOXJJTN5JTY5E2V2M6YCGH3YROMDPAVCNFSM5GJKL3WKU5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TCOJSGEZTMMZQHEZA>.
You are receiving this because you authored the thread.Message ID: ***@***.******@***.***>>
|
Thank you for the quick reply. Can you please help us understand the reason for the change? I couldn't find an open issue for this particular feature. This would help us better triage and prioritize the PRs and their scopes. Thank you. |
In my project I can`t use connection pool, only dedicated connection. And I already use named parameters for queries.
It is the reason to make this PR. Right now I use my fork in project. If PR will be merged – I`ll switch to original sqlx package.
From: Florin Pățan ***@***.***>
Sent: Friday, February 2, 2024 10:13 AM
To: jmoiron/sqlx ***@***.***>
Cc: Козлов Евгений ***@***.***>; Author ***@***.***>
Subject: Re: [jmoiron/sqlx] Named query for Conn (PR #780)
Thank you for the quick reply. Can you please help us understand the reason for the change?
I couldn't find an open issue for this particular feature.
This would help us better triage and prioritize the PRs and their scopes.
Thank you.
—
Reply to this email directly, view it on GitHub<#780 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AOXJJTIUIASG4APX4RLVFPDYRSGYVAVCNFSM5GJKL3WKU5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TCOJSGMYTKMRZHA2A>.
You are receiving this because you authored the thread.Message ID: ***@***.******@***.***>>
|
Named query for Conn.