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