Mismatch between uuid bind to as variable vs value returned from db with jdbc

[solita-kattelus]

What happens?

Testing new UUID binding and somehow get mismatching UUID back

    @Test
    public void testUUIDBinding() throws SQLException {
        UUID uuid = UUID.fromString("0b17ce61-375c-4ad8-97b3-349d96d35ab1");
        Connection conn = DriverManager.getConnection("jdbc:duckdb:");
        PreparedStatement statement = conn.prepareStatement("select ? as value");
        statement.setObject(1, uuid);
        ResultSet resultSet = statement.executeQuery();
        Assertions.assertTrue(resultSet.next());
        Object object = resultSet.getObject(1);
        Assertions.assertEquals(uuid, object);
        Assertions.assertFalse(resultSet.next());
    }

And it fails with Assertions.assertEquals(uuid, object); check as:

Expected :0b17ce61-375c-4ad8-97b3-349d96d35ab1
Actual   :8b17ce61-375c-4ad8-97b3-349d96d35ab1

To Reproduce

Reproduce can be done with junit test above.

OS:

macOs arm64

DuckDB Version:

0.10.0

DuckDB Client:

JDBC

Full Name:

Marko Kattelus

Affiliation:

Solita Oy

Have you tried this on the latest nightly build?

I have tested with a release build (and could not test with a nightly build)

Have you tried the steps to reproduce? Do they include all relevant data and configuration? Does the issue you report still appear there?

• Yes, I have

[rsmckinney]

Just ran into the same problem. Ouch. Is there a fix underway?

[github-actions]

This issue is stale because it has been open 90 days with no activity. Remove stale label or comment or this will be closed in 30 days.

[github-actions]

This issue was closed because it has been stale for 30 days with no activity.

[rsmckinney]

This is a bit too severe to ignore, no?

[staticlibs]

This should be fixed in #160 , basically DuckDB modifies the UUID when parsing it from string literals. And, for consistency, the same logic needs to be replicated when passing bind variables from Java.

[guizmaii]

@staticlibs Are you guys publishing snapshot versions? Not sure we wanna wait for v1.2.2, scheduled in April, for having access to this fix

[staticlibs]

@guizmaii

Nightly builds (that are expected to be done from the latest main) are available here, but the PR needs to be merged first to get there. I will ask around tomorrow if it is possible to expedite JDBC PRs reviews.