Github user squito commented on the issue:

    https://github.com/apache/spark/pull/19702
  
    hey thanks for doing this @cloud-fan but I have a small request -- can we 
get another day to review how this works, especially in connection with 
somewhat recent changes in parquet to include a 
[`isAdjustedToUTC`](https://github.com/apache/parquet-format/blob/master/src/main/thrift/parquet.thrift#L271)?
  Just want to make sure this doesn't cause problems with resolving with / 
without time zone in parquet data later on.  (don't think it should, just want 
to take a bit closer look)


---

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

Reply via email to