Parameter parsing for PreparedStatement about hazelcast-jdbc HOT 4 OPEN

IMO this should be handled by server and that it already is. The driver should send the sql string and params as is to the server.

from hazelcast-jdbc.

@viliam-durina according to the JavaDoc of setter methods it should validate the parameters. For example for the query SELECT * FROM foo WHERE bar=? calling statement.setInt(2, 3) should throw an exception. However, the JDBC is calling the server on execute after setting the parameter. We decided not to implement it for now, as won't bring much value but will require a lot of effort. But we may need to implement it in the future

from hazelcast-jdbc.

I tested postgresql:

PreparedStatement st = conn.prepareStatement("select * from t");
st.setInt(1, 1); // throws

st = conn.prepareStatement("select * from ?");
st.setInt(1, 1); // does not throw

st = conn.prepareStatement("select * from m /* where a=? */");
st.setInt(1, 1); // throws

// this fails at parsing
st = conn.prepareStatement("select * from m /* where a=?");

This is their parser: org.postgresql.core.Parser.parseSql(), it's quite large, but it's still best-effort, see the 2nd query. IMO it's fine to always throw in execute.

from hazelcast-jdbc.

The second one should also throw an exception at the execute. Agree that we don't need to invest in it, at least for now

from hazelcast-jdbc.