sqlx icon indicating copy to clipboard operation
sqlx copied to clipboard

Named query for Conn

Open ekozlov-search opened this issue 4 years ago • 4 comments

Named query for Conn.

ekozlov-search avatar Oct 19 '21 14:10 ekozlov-search

Hello, @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.

dlsniper avatar Feb 01 '24 13:02 dlsniper

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-bkennedyhttps://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 GitHubhttps://github.com/jmoiron/sqlx/pull/780#issuecomment-1921363092, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AOXJJTN5JTY5E2V2M6YCGH3YROMDPAVCNFSM5GJKL3WKU5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TCOJSGEZTMMZQHEZA. You are receiving this because you authored the thread.Message ID: @.@.>>

ekozlov-search avatar Feb 02 '24 05:02 ekozlov-search

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.

dlsniper avatar Feb 02 '24 07:02 dlsniper

In my project I cant 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 – Ill 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 GitHubhttps://github.com/jmoiron/sqlx/pull/780#issuecomment-1923152984, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AOXJJTIUIASG4APX4RLVFPDYRSGYVAVCNFSM5GJKL3WKU5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TCOJSGMYTKMRZHA2A. You are receiving this because you authored the thread.Message ID: @.@.>>

ekozlov-search avatar Feb 07 '24 10:02 ekozlov-search