[GitHub] [spark] c21 commented on a change in pull request #30138: [SPARK-33075][SQL] Enable auto bucketed scan by default (disable only for cached query)

c21 commented on a change in pull request #30138: URL: https://github.com/apache/spark/pull/30138#discussion_r628711441 ## File path: sql/core/src/main/scala/org/apache/spark/sql/execution/adaptive/AdaptiveSparkPlanHelper.scala ## @@ -139,15 +138,21 @@ trait AdaptiveSparkPlanH

[GitHub] [spark] c21 commented on a change in pull request #30138: [SPARK-33075][SQL] Enable auto bucketed scan by default (disable only for cached query)

c21 commented on a change in pull request #30138: URL: https://github.com/apache/spark/pull/30138#discussion_r628695013 ## File path: sql/core/src/main/scala/org/apache/spark/sql/execution/adaptive/AdaptiveSparkPlanHelper.scala ## @@ -139,15 +138,21 @@ trait AdaptiveSparkPlanH

[GitHub] [spark] c21 commented on a change in pull request #30138: [SPARK-33075][SQL] Enable auto bucketed scan by default (disable only for cached query)

c21 commented on a change in pull request #30138: URL: https://github.com/apache/spark/pull/30138#discussion_r511757506 ## File path: sql/core/src/main/scala/org/apache/spark/sql/SparkSession.scala ## @@ -1077,6 +1077,27 @@ object SparkSession extends Logging { throw new

[GitHub] [spark] c21 commented on a change in pull request #30138: [SPARK-33075][SQL] Enable auto bucketed scan by default (disable only for cached query)

c21 commented on a change in pull request #30138: URL: https://github.com/apache/spark/pull/30138#discussion_r511747673 ## File path: sql/core/src/main/scala/org/apache/spark/sql/execution/CacheManager.scala ## @@ -79,10 +92,11 @@ class CacheManager extends Logging with Adapt

[GitHub] [spark] c21 commented on a change in pull request #30138: [SPARK-33075][SQL] Enable auto bucketed scan by default (disable only for cached query)

c21 commented on a change in pull request #30138: URL: https://github.com/apache/spark/pull/30138#discussion_r511321724 ## File path: sql/core/src/main/scala/org/apache/spark/sql/SparkSession.scala ## @@ -1077,6 +1077,27 @@ object SparkSession extends Logging { throw new

[GitHub] [spark] c21 commented on a change in pull request #30138: [SPARK-33075][SQL] Enable auto bucketed scan by default (disable only for cached query)

c21 commented on a change in pull request #30138: URL: https://github.com/apache/spark/pull/30138#discussion_r511319323 ## File path: sql/core/src/main/scala/org/apache/spark/sql/SparkSession.scala ## @@ -1077,6 +1077,27 @@ object SparkSession extends Logging { throw new

[GitHub] [spark] c21 commented on a change in pull request #30138: [SPARK-33075][SQL] Enable auto bucketed scan by default (disable only for cached query)

c21 commented on a change in pull request #30138: URL: https://github.com/apache/spark/pull/30138#discussion_r511167105 ## File path: sql/core/src/main/scala/org/apache/spark/sql/execution/adaptive/AdaptiveSparkPlanHelper.scala ## @@ -139,15 +138,21 @@ trait AdaptiveSparkPlanH

[GitHub] [spark] c21 commented on a change in pull request #30138: [SPARK-33075][SQL] Enable auto bucketed scan by default (disable only for cached query)

c21 commented on a change in pull request #30138: URL: https://github.com/apache/spark/pull/30138#discussion_r510736242 ## File path: sql/core/src/test/scala/org/apache/spark/sql/sources/DisableUnnecessaryBucketedScanSuite.scala ## @@ -218,4 +221,27 @@ abstract class DisableUn