-
Notifications
You must be signed in to change notification settings - Fork 414
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
fix: use prepared stmt in sql #3100
Conversation
Signed-off-by: Song Gao <disxiaofei@163.com>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #3100 +/- ##
==========================================
- Coverage 68.14% 68.12% -0.02%
==========================================
Files 391 391
Lines 37014 37031 +17
==========================================
+ Hits 25221 25227 +6
- Misses 9547 9553 +6
- Partials 2246 2251 +5 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need to change the SQL plugin as well?
I think it's not necessary, the query is generated by kuiperd itself. |
Signed-off-by: Song Gao <disxiaofei@163.com>
Signed-off-by: Song Gao <disxiaofei@163.com>
Signed-off-by: Song Gao <disxiaofei@163.com>
cherry-picked from commit 1a9c745
cherry-picked from commit 1a9c745 Signed-off-by: Yiheng Cao <65160922+Crispy-fried-chicken@users.noreply.github.com>
cherry-picked from commit 1a9c745 Signed-off-by: Yiheng Cao <65160922+Crispy-fried-chicken@users.noreply.github.com>
cherry-picked from commit 1a9c745
cherry-picked from commit 1a9c745 Signed-off-by: Yiheng Cao <65160922+Crispy-fried-chicken@users.noreply.github.com>
No description provided.