Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Rewrite StdDateFormat ISO-8601 handling functionality #1678

Closed
cowtowncoder opened this issue Jun 24, 2017 · 0 comments
Closed

Rewrite StdDateFormat ISO-8601 handling functionality #1678

cowtowncoder opened this issue Jun 24, 2017 · 0 comments
Milestone
2.9.0

Comments

@cowtowncoder
Copy link
Member

cowtowncoder commented Jun 24, 2017 •
edited

StdDateFormat is pretty fragile implementation, but especially so its ISO-8601 decoder part.
Problems range from "interesting" choice of valid values to concurrency/performance problems (that is, how to balance access to SimpleDateFormat). But looking at SimpleDateFormat and Calendar, real calculations in latter, not former; plus, we don't really need flexibility of SimpleDateFormat here (although for RFC1123 we may still want it).

And it probably also makes sense to rewrite ISO-8601 serialization parts while we're at it.
@cowtowncoder cowtowncoder added this to the 2.9.0 milestone Jun 25, 2017
@cowtowncoder cowtowncoder mentioned this issue Jun 25, 2017
Closed
@cowtowncoder cowtowncoder changed the title Rewrite StdDateFormat ISO-8601 deserialization functionality Rewrite StdDateFormat ISO-8601 handling functionality Jun 25, 2017
cowtowncoder added a commit that referenced this issue Jun 25, 2017
@cowtowncoder
Bit more work on #1678, serialization side
37b780b
@cowtowncoder cowtowncoder mentioned this issue Jun 27, 2017
Closed
@huxi huxi mentioned this issue Oct 5, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
2.9.0
Development

No branches or pull requests
1 participant
@cowtowncoder