Github user squito commented on the issue:

    https://github.com/apache/spark/pull/16781
  
    pinging some potential reviewers:
    @liancheng @yhuai for prior work in hive compatibility
    @ueshin for work on timezone support
    
    Note that the timezone support from SPARK-18350 is to some extent at odds 
with this.   Its not a total conflict, because that is addressing how spark 
stores data in its own json & csv format (among others).  But this is about 
making sure that *hive* tables are consistent between different formats.  I am 
somewhat puzzled by those changes, as they seem to be changing the meaning of 
the "TIMESTAMP" datatype to be more like "TIMESTAMP WITH TIMEZONE" in other sql 
engines, but I'll also admit I'm out of my expertise here.
    
    This is not the area of code I am most familiar with, happy to get 
feedback.  One thing in particular I haven't addressed is predicate pushdown -- 
in particular, I'm not sure how to *test* predicate pushdown.  I can poke at 
that some more, but I'd appreciate any guidance.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

Reply via email to