feat: add ability to not prepare statements#217
feat: add ability to not prepare statements#217burningalchemist merged 2 commits intoburningalchemist:masterfrom fffinkel:no-prepared-statement
Conversation
|
Hi @fffinkel, thanks for your PR! Could you please share a bit more details about the change? It's just for the history and better visibility of the PR. Just a few lines with an example would be perfect! :) |
|
Hey @burningalchemist, sorry for such a delay in response! We added this feature because we connect to ProxySQL, which does not properly support prepared statements. |
|
Hey @fffinkel, no worries. Got it, thanks. 👍 I need to fix one of the bugs first, but then I'll prioritize this PR. I just wanted to learn more about the use case to test it as well (and eventually cover everything with proper tests once I have a bit more time). |
|
Thanks very much! |
|
Oh wow, this is so timely @fffinkel, I was testing out this exporter right now for exactly the ProxySQL use case and ran into this problem and was considering whipping out a PR when I noticed this! Thanks for doing this. |
|
any chance we can get a release on this? |
|
Hey @srikiraju! Apologies, I haven't received the notification to the message. I'm preparing a new release next week. 👍 Will keep you posted. |
Add this for cases when the backend does not support it