[GitHub] [spark] maropu commented on pull request #29804: [SPARK-32859][SQL] Introduce physical rule to decide bucketing dynamically

maropu commented on pull request #29804: URL: https://github.com/apache/spark/pull/29804#issuecomment-702457642 Thanks, @c21 @viirya ! Merged to master. This is an automated message from the Apache Git Service. To respond to

[GitHub] [spark] maropu commented on pull request #29804: [SPARK-32859][SQL] Introduce physical rule to decide bucketing dynamically

maropu commented on pull request #29804: URL: https://github.com/apache/spark/pull/29804#issuecomment-702456924 okay, it looks okay as a first step for this feature, I think. Thanks for the valuable work, @c21! I'll merge this. If more comments are given by @cloud-fan, please address them

[GitHub] [spark] maropu commented on pull request #29804: [SPARK-32859][SQL] Introduce physical rule to decide bucketing dynamically

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?

[GitHub] [spark] maropu commented on pull request #29804: [SPARK-32859][SQL] Introduce physical rule to decide bucketing dynamically

maropu commented on pull request #29804: URL: https://github.com/apache/spark/pull/29804#issuecomment-695134989 Looks an interesting idea, @c21 ! This is an automated message from the Apache Git Service. To respond to the mes