zhangbutao commented on PR #5868:
URL: https://github.com/apache/hive/pull/5868#issuecomment-3000894272

   > @ayushtkn, @zhangbutao and @deniskuzZ . This change seems to be needed for 
hive 4 upgrade in spark, can we please included this in 4.1.0 ?
   > 
   > Thanks!
   
   @simhadri-g I don't quite understand the context of this PR. Will this issue 
block the upgrade of Hive4 dependencies in Spark?
   
   From the description `While Hive assumes that isUTC is always true, Spark 
uses local time zone`, I'm not sure if this can be defined as a problem. 
   BTW, Can this issue be fixed from the Spark side?
   
   I'm not very familiar with this piece of code. If someone has a thorough 
understanding of the fix, i am ok to include it to 4.1.0. But  personally i 
prefer to incorporate this code into the next version to avoid any possible 
impact for the release 4.1.0.
   
   Thanks.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


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

Reply via email to