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.

Sign up for GitHub

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

Jump to bottom

use setLong instead of setInt #428

Merged

kortemik merged 1 commit into teragrep:main from elliVM:preparedstatement-long

Nov 21, 2024

Contributor

elliVM commented Nov 21, 2024

Fix prepared statements that incorrectly used setInt() instead of setLong()
Update test case to only pass when setLong is used


          use setLong instead of setInt and update test for partition values la…

3020cdf

…rger than max int

kortemik approved these changes

View reviewed changes

kortemik merged commit ed694e8 into teragrep:main

1 check passed

elliVM self-assigned this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet