[
https://issues.apache.org/jira/browse/ORC-306?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16378042#comment-16378042
]
ASF GitHub Bot commented on ORC-306:
------------------------------------
GitHub user omalley opened a pull request:
https://github.com/apache/orc/pull/220
ORC-306 Correct pre-1970 timestamps that were off by one second.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/omalley/orc orc-306
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/orc/pull/220.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #220
----
commit 1f38e1d58fe012dcf850ff1c0ba5d887ca0e21e4
Author: Owen O'Malley <omalley@...>
Date: 2018-02-26T23:27:52Z
ORC-306 Correct pre-1970 timestamps that were off by one second.
----
> Fix incorrect workaround for bug in java.sql.Timestamp
> ------------------------------------------------------
>
> Key: ORC-306
> URL: https://issues.apache.org/jira/browse/ORC-306
> Project: ORC
> Issue Type: Test
> Reporter: Owen O'Malley
> Assignee: Owen O'Malley
> Priority: Major
>
> There is a bug in java.sql.Timestamp where if the timestamp is of the form
> "YYYY-MM-DD HH:MM:SS.000XXX" and year is before 1970, the resulting time is
> off by one second. Unfortunately, the ORC reader was unintentionally
> triggering this case for all timestamps before 1970 and thus included
> compensating code to correct the problem.
> Unfortunately, the workaround means that the user get incorrect results if
> they don't use Timestamp to interpret the results.*1900-05-05 12:34:56.1* is
> stored in TimestampColumnVector as -2198201103900 millis and 100000000 nanos.
> The ORC reader incorrectly restores that as -2198201103000 and 100000000
> (notice the missing 900 millis), then adjusts the millis to -2198201102000.
> Since the last three digits of the millis are 0 that triggers the Timestamp
> bug and becomes the matching *1900-05-05 12:34:56.1*.
> So basically, ORC added a bug to compensate for Java's bug.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
