[GitHub] [spark] dongjoon-hyun commented on a diff in pull request #40390: [SPARK-42768][SQL] Enable cached plan apply AQE by default

2023-08-21 Thread via GitHub
dongjoon-hyun commented on code in PR #40390: URL: https://github.com/apache/spark/pull/40390#discussion_r1300808158 ## sql/core/src/main/scala/org/apache/spark/sql/execution/CacheManager.scala: ## @@ -395,7 +395,10 @@ class CacheManager extends Logging with

[GitHub] [spark] dongjoon-hyun commented on a diff in pull request #40390: [SPARK-42768][SQL] Enable cached plan apply AQE by default

2023-08-21 Thread via GitHub
dongjoon-hyun commented on code in PR #40390: URL: https://github.com/apache/spark/pull/40390#discussion_r1299657447 ## sql/core/src/main/scala/org/apache/spark/sql/execution/CacheManager.scala: ## @@ -395,7 +395,10 @@ class CacheManager extends Logging with

[GitHub] [spark] dongjoon-hyun commented on a diff in pull request #40390: [SPARK-42768][SQL] Enable cached plan apply AQE by default

2023-03-12 Thread via GitHub
dongjoon-hyun commented on code in PR #40390: URL: https://github.com/apache/spark/pull/40390#discussion_r1133456778 ## sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala: ## @@ -1493,15 +1493,14 @@ object SQLConf { val