Merge pull request #326 from joker1007/fix-timestamp-casting

fix: fix double casting on `getTimestamp`
embulk · Oct 3, 2023 · c2acb83 · c2acb83
2 parents ecec67b + 318f97f
commit c2acb83
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/embulk-output-jdbc/src/main/java/org/embulk/output/jdbc/MemoryRecord.java b/embulk-output-jdbc/src/main/java/org/embulk/output/jdbc/MemoryRecord.java
@@ -41,7 +41,7 @@ public String getString(Column column)
 
     public Instant getTimestamp(Column column)
     {
-        return ((org.embulk.spi.time.Timestamp) getValue(column)).getInstant();
+        return (Instant)getValue(column);
     }
 
     public Value getJson(Column column)

diff --git a/embulk-output-jdbc/src/main/java/org/embulk/output/jdbc/PageReaderRecord.java b/embulk-output-jdbc/src/main/java/org/embulk/output/jdbc/PageReaderRecord.java
@@ -22,7 +22,7 @@ public class PageReaderRecord implements Record
     public PageReaderRecord(PageReader pageReader)
     {
         this.pageReader = pageReader;
-        readRecords = new ArrayList<MemoryRecord>();
+        readRecords = new ArrayList<>();
     }
 
     public void setPage(Page page)