Github user wgtmac commented on a diff in the pull request:
https://github.com/apache/orc/pull/249#discussion_r185691649
--- Diff:
java/core/src/java/org/apache/orc/impl/writer/TimestampTreeWriter.java ---
@@ -28,7 +28,9 @@
import org.apache.orc.impl.SerializationUtils;
import java.io.IOException;
-import java.sql.Timestamp;
+import java.text.DateFormat;
+import java.text.ParseException;
+import java.text.SimpleDateFormat;
import java.util.TimeZone;
public class TimestampTreeWriter extends TreeWriterBase {
--- End diff --
We should also change writeBatch function below.
The input vector.isUTC may be true while writer.isUseUTCTimestamp() is
false; vice versa. In this case, we need to convert them to correct writer
timezone.
---
