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
Add Prepared Statement support #349
Add Prepared Statement support #349
Conversation
Great work @guyboertje - left a few minor comments but otherwise LGTM! |
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.
Cool stuff @guyboertje - comments are mostly nits
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.
LGTM - happy with the changes that you've made!
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.
LGTM
Co-Authored-By: Colin Surprenant <colin.surprenant@gmail.com>
Co-Authored-By: Rob Bavey <rob.bavey@elastic.co>
293d974
to
ec8600b
Compare
Code, tests, docs and version bump.