Github user dongjoon-hyun commented on a diff in the pull request:

    https://github.com/apache/spark/pull/21372#discussion_r189674371
  
    --- Diff: 
sql/core/src/main/java/org/apache/spark/sql/execution/datasources/orc/OrcColumnVector.java
 ---
    @@ -136,7 +136,7 @@ public int getInt(int rowId) {
       public long getLong(int rowId) {
         int index = getRowIndex(rowId);
         if (isTimestamp) {
    -      return timestampData.time[index] * 1000 + timestampData.nanos[index] 
/ 1000;
    +      return timestampData.time[index] * 1000 + timestampData.nanos[index] 
/ 1000 % 1000;
    --- End diff --
    
    `OrcHadoopFsRelationSuite` covers this changes via end-to-end write and 
read test cases.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to