klcopp commented on pull request #1501: URL: https://github.com/apache/hive/pull/1501#issuecomment-694340954
@szlta What you say makes sense. The only issue being that if we have one session-level config controlling the cache, and we use it to always turn off the cache during compaction, we'll never allow turning it back on during compaction – this fix wouldn't be behind a feature flag. I'm tentatively okay with this – @zchovan are you ok with this? ---------------------------------------------------------------- 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. 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
