-
Notifications
You must be signed in to change notification settings - Fork 16
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
Support PreparedStatement.setNull for jdbc #17
Comments
@dwayneberry should we make the suggested change? |
FWIW, I compiled it myself (I had to update the |
Well, I was indeed missing something. This does not work for batch inserts (e.g. it will insert 'NULL' as string value for text columns). |
@RalphLoen Any comment on this? You've been playing with Spark. |
@poroszd Currently this code does not differentiate between empty string and NULL. It treats empty string as NULL. To get your fix to work
Should do it. But really the longer term solution, if you want to be able to support an empty length string (edge case this fix will not support), we will need to add an additional flag |
I'm using the jdbc interface to move data from Spark to MapD, but it does not work for
null
values, asMapDPreparedStatement.setNull()
is not implemented.Am I missing something, or it would be really this simple?
The text was updated successfully, but these errors were encountered: