| This appears to be a simple fix in BindingTypeHelper: the class contains a very nice case statement that routes the three TemporalTypes to a different method; unfortunately, each of these methods does the same thing when the "javaType" is "java.util.Date", i.e. the methods all return "TimestampType.INSTANCE" when they should return "TimestampType.INSTANCE", "DateType.INSTANCE", and "TimeType.INSTANCE" respectively. Can this please be addressed? The BindingTypeHelper class even contains a comment: "todo : (5.2) review Java type handling for sanity. This part was done quickly" which implies that the existing logic was not well thought out and should be reviewed/fixed sooner than later. |