[jira] [Comment Edited] (TORQUE-375) Support for java.time API

    [ https://issues.apache.org/jira/browse/TORQUE-375?page=3Dcom.atlassian=
.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=3D1809=
1842#comment-18091842 ]=20

Georg Kallidis edited comment on TORQUE-375 at 6/26/26 12:12 PM:
-----------------------------------------------------------------

I found this comment in testJavaDefault:
{code:java}
For date values, java fields are null if the default is read{code}
This is not a contract, and it is already initialized, when CURRENT_DATE is=
 the default as for hsqldb with:
default=3D"CURRENT_DATE"=20
with the patches in default-value-schema.sql for e.g. table database_defaul=
t_values (in date_field DATE default '2010-09-08', BaseDatabaseDefaultValue=
s private java.time.LocalDate dateField =3D LocalDate.parse("2010-09-08");

While in the curent trunk it is database database_default_values, date_fiel=
d DATE default '2010-09-08'=C2=A0 (that is the same), but BaseDatabaseDefau=
ltValues private java.util.Date dateField =3D null;

This seems wrong (?).

We should change the test, as fields may be null only if not with a default=
. This would resolve at least the first two test errors ...


was (Author: gk):
a comment in testJavaDefault:

=C2=A0
{code:java}
For date values, java fields are null if the default is read{code}
=C2=A0

This is not a contract, and it is already initialized, when CURRENT_DATE is=
 the default as for hsqldb with:
default=3D"CURRENT_DATE"=20
with the patches in default-value-schema.sql for e.g. table database_defaul=
t_values (in date_field DATE default '2010-09-08', BaseDatabaseDefaultValue=
s private java.time.LocalDate dateField =3D LocalDate.parse("2010-09-08");

While in the curent trunk it is database database_default_values, date_fiel=
d DATE default '2010-09-08'=C2=A0 (that is the same), but BaseDatabaseDefau=
ltValues private java.util.Date dateField =3D null;

This seems wrong (?).

We should change the test, as fields may be null only if not with a default=
. This would resolve at least the first two test errors ...

> Support for java.time API
> -------------------------
>
>                 Key: TORQUE-375
>                 URL: https://issues.apache.org/jira/browse/TORQUE-375
>             Project: Torque
>          Issue Type: Improvement
>    Affects Versions: 7.1
>            Reporter: Graham Leggett
>            Priority: Major
>         Attachments: torque8-javatime1.patch, torque8-javatime2.patch, to=
rque8-javatime3.patch, torque8-javatime4.patch
>
>
> Switch out java.util.Date, replace with the java.time API.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)