-
Notifications
You must be signed in to change notification settings - Fork 2.4k
Support Java 8 Dates for Job Parameters #1035
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
Hi @benas, Please assign me the task, I am starting work on this one. |
Hi @parikshitdutta , Thank you for your offer to help.
We have a feature request to use timezone aware datatypes (see #871), so I'm not sure Please note that we did not start working on v5 yet, we just assigned a few issues we think should be addressed in a major version to the v5.0.0 milestone. |
Thanks @benas for sharing the information. |
This commit also changes the way job parameters are parsed and persisted. NB: This commit should ideally have been split into two change sets. But the changes are tightly related that is was not possible to separate them. Related to: * spring-projects#3960 * spring-projects#2122 * spring-projects#1035 * spring-projects#1983
This commit also changes the way job parameters are parsed and persisted. NB: This commit should ideally have been split into two change sets. But the changes are tightly related that is was not possible to separate them. Related to: * spring-projects#3960 * spring-projects#2122 * spring-projects#1035 * spring-projects#1983
This commit also changes the way job parameters are parsed and persisted. NB: This commit should ideally have been split into two change sets. But the changes are tightly related that is was not possible to separate them. Related to: * spring-projects#3960 * spring-projects#2122 * spring-projects#1035 * spring-projects#1983
@parikshitdutta The impact of #871 was evaluated and the feature was deferred to a future version, see the rationale behind this decision in #871 (comment) and #1014 (comment). Hence, the type of timestamp fields was changed to |
Philippe Marschall opened BATCH-2569 and commented
It would be cool if
JobParameter
would support the Java 8 Date and Time API. Since the value on the table is of SQL typeTIMESTAMP
the corresponding type would beLocalDateTime
.Unfortunately due the design of the
JobParameter
class with a single#getValue()
method doing this would break backward compatibility.One way to avoid this would be a. new
getLocalDate()
method.No further details from BATCH-2569
The text was updated successfully, but these errors were encountered: