maropu commented on pull request #29804: URL: https://github.com/apache/spark/pull/29804#issuecomment-701203694
> I am fine with disabling the feature by default, and let user opt-in case by case. As normally for SQL users (not dataframe), the cache query should not be very popular. WDYT? @maropu . thanks. Ah, I see and I missed the case. The @viirya suggestion makes sense, so disabling it by default looks okay to me. Could you drop `.internal` accordingly if users need to care about the added option? ---------------------------------------------------------------- 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: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
