ayushtkn commented on PR #4793:
URL: https://github.com/apache/hive/pull/4793#issuecomment-1763370610

   I have asked the spark folks to check on this, not sure how spark is 
working. This config stuff came from that side only :-) 
   
   Though I am still not sure, if we don't set this config & if spark uses old 
code, if the table property isn't set here:
   
https://github.com/apache/iceberg/blob/main/hive-metastore/src/main/java/org/apache/iceberg/hive/HiveTableOperations.java#L595-L596
   
   it will still fallback on the config & if false, it will not set these 
params, so we should be safe & sorted.


-- 
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