Attila Jeges has posted comments on this change. Change subject: Revert "IMPALA-2716: Hive/Impala incompatibility for timestamp data in Parquet" ......................................................................
Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/6896/1/be/src/runtime/timestamp-value.h File be/src/runtime/timestamp-value.h: Line 25: #include <ctime> > why do we need these headers? Removed the new includes and kept boost/date_time/local_time/local_time.hpp (it is required for boost::posix_time) http://gerrit.cloudera.org:8080/#/c/6896/1/fe/src/main/java/org/apache/impala/catalog/HdfsTable.java File fe/src/main/java/org/apache/impala/catalog/HdfsTable.java: Line 123: // Avro schema of this table if this is an Avro table, otherwise null. Set in load(). > what did these have to do with your change? are they just dead variables (i Removed them, they were left behind accidentaly when I resolved the conflicts. -- To view, visit http://gerrit.cloudera.org:8080/6896 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic66de277c622748540c1b9969152c2cabed1f3bd Gerrit-PatchSet: 2 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Attila Jeges <[email protected]> Gerrit-Reviewer: Attila Jeges <[email protected]> Gerrit-Reviewer: Dan Hecht <[email protected]> Gerrit-HasComments: Yes
