[ https://issues.apache.org/jira/browse/ORC-346?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16444485#comment-16444485 ]
rip.nsk edited comment on ORC-346 at 4/19/18 6:01 PM: ------------------------------------------------------ TEST(Writer, writeTimestamp) failed with the following change: +c++/test/TestWriter.cc:573 -- time_t currTime = std::time(nullptr); + time_t currTime = -14210715; // 1969-07-20 12:34:45 was (Author: rip....@gmail.com): TEST(Writer, writeTimestamp) failed with the following change: +c++/test/TestWriter.cc:573 --- time_t currTime = std::time(nullptr); +++ time_t currTime = -14210715; // 1969-07-20 12:34:45 > Bug in TimestampColumnReader > ---------------------------- > > Key: ORC-346 > URL: https://issues.apache.org/jira/browse/ORC-346 > Project: ORC > Issue Type: Bug > Components: C++ > Reporter: rip.nsk > Priority: Critical > > void TimestampColumnReader::next(ColumnVectorBatch& rowBatch, uint64_t > numValues, char *notNull) > has the following code: > c++\src\ColumnReader.cc:338 > int64_t writerTime = secsBuffer[i] + epochOffset; > secsBuffer[i] = writerTimezone.convertToUTC(writerTime); > {color:#f79232}if (secsBuffer[i] < 0 && nanoBuffer[i] != 0) {{color} > {color:#f79232} secsBuffer[i] -= 1;{color} > {color:#f79232} }{color} > {color:#333333}which likely leads to read wrong seconds value for PRE_1970 > dates{color} -- This message was sent by Atlassian JIRA (v7.6.3#76005)