org.jooq.tools.Convert should delegate to org.jooq.impl.Convert, instead of duplicating implementations - Merge [#16044]

[lukaseder]

No description provided.

[alf]

This change seems to have fixed a bug in the handling of LocalDate from within a multiset (i.e. json).

We have a green test that turns red when bumping jOOQ from 3.19.1 to 3.19.2 (and further). This has held us from upgrading since we didn't have time to look into the problem before now. I've now simplified the failing test to the following:

var result = ctx.select(
                multiset(select(
                        val("2013-07-01T00:00:00", LocalDate.class)
                        )
                )
        ).fetchOne();

        assertNull(result.component1().get(0).component1());

Obviously the test is incorrect, but it slipped past our QA.

[lukaseder]

@alf I'd say, this was an expected improvement, given that this issue of yours has been fixed in 3.19.2?

• DATE column in Oracle is null when fetched through multiset #16043

[alf]

Indeed. Just too many things going on so I can’t quite keep up. ❤️

[lukaseder]

No worries :)