Github user wgtmac commented on a diff in the pull request:

    https://github.com/apache/orc/pull/18#discussion_r175328447
  
    --- Diff: c++/src/ColumnReader.cc ---
    @@ -327,7 +329,9 @@ namespace orc {
                 nanoBuffer[i] *= 10;
               }
             }
    -        secsBuffer[i] += epochOffset;
    +        int64_t writerTime = secsBuffer[i] + epochOffset;
    +        secsBuffer[i] = writerTime +
    --- End diff --
    
    @omalley I think here introduces an inconsistency between reader and 
writer. The secsBuffer[i] in the TimestampVectorBatch of reader side will not 
get same value on the java writer/reader side.


---

Reply via email to