Now able to avoid duplication in InstantConverter and LocalDateTimeConverter

Author: vslee

JSqlServerBulkInsert/src/main/java/de/bytefish/jsqlserverbulkinsert/converters/InstantConverter.java

@@ -7,11 +7,6 @@ public class InstantConverter extends BaseConverter<Instant> {
 
     @Override
     public Object internalConvert(Instant value) {
-
-        if (value == null) {
-            return null;
-        }
-
         Timestamp castedResult = new Timestamp(value.toEpochMilli());
 
         // Round to the nearest 100 nanoseconds, the precision that SQL server can handle:

JSqlServerBulkInsert/src/main/java/de/bytefish/jsqlserverbulkinsert/converters/LocalDateTimeConverter.java

@@ -4,12 +4,13 @@
 import java.time.LocalDate;
 import java.time.LocalDateTime;
 import java.time.OffsetDateTime;
+import java.time.ZoneOffset;
 
 public class LocalDateTimeConverter extends BaseConverter<LocalDateTime> {
+    static ZoneOffset zoneOffset =  OffsetDateTime.now().getOffset();
     @Override
     public Object internalConvert(LocalDateTime value) {
-
-        long epochSeconds = value.toEpochSecond(OffsetDateTime.now().getOffset());
+        long epochSeconds = value.toEpochSecond(zoneOffset);
 
         // Convert to milliseconds to create the Timestamp:
         Timestamp castedResult = new Timestamp(epochSeconds * 1000);