Re: [PR] [SPARK-50903][CONNECT] Cache logical plans after analysis [spark]

2025-02-13 Thread via GitHub
changgyoopark-db commented on code in PR #49584: URL: https://github.com/apache/spark/pull/49584#discussion_r1954742185 ## sql/connect/server/src/main/scala/org/apache/spark/sql/connect/service/SessionHolder.scala: ## @@ -440,46 +443,64 @@ case class SessionHolder(userId: String

Re: [PR] [SPARK-50903][CONNECT] Cache logical plans after analysis [spark]

2025-01-30 Thread via GitHub
changgyoopark-db commented on code in PR #49584: URL: https://github.com/apache/spark/pull/49584#discussion_r1936816392 ## sql/connect/server/src/main/scala/org/apache/spark/sql/connect/service/SessionHolder.scala: ## @@ -440,46 +443,64 @@ case class SessionHolder(userId: String

Re: [PR] [SPARK-50903][CONNECT] Cache logical plans after analysis [spark]

2025-01-30 Thread via GitHub
changgyoopark-db commented on code in PR #49584: URL: https://github.com/apache/spark/pull/49584#discussion_r1936815612 ## sql/connect/server/src/main/scala/org/apache/spark/sql/connect/service/SessionHolder.scala: ## @@ -440,46 +443,64 @@ case class SessionHolder(userId: String

Re: [PR] [SPARK-50903][CONNECT] Cache logical plans after analysis [spark]

2025-01-30 Thread via GitHub
hvanhovell commented on code in PR #49584: URL: https://github.com/apache/spark/pull/49584#discussion_r1936138008 ## sql/connect/server/src/main/scala/org/apache/spark/sql/connect/service/SessionHolder.scala: ## @@ -440,46 +443,64 @@ case class SessionHolder(userId: String, sess

Re: [PR] [SPARK-50903][CONNECT] Cache logical plans after analysis [spark]

2025-01-30 Thread via GitHub
hvanhovell commented on code in PR #49584: URL: https://github.com/apache/spark/pull/49584#discussion_r1936121699 ## sql/connect/server/src/main/scala/org/apache/spark/sql/connect/service/SessionHolder.scala: ## @@ -440,46 +443,64 @@ case class SessionHolder(userId: String, sess

Re: [PR] [SPARK-50903][CONNECT] Cache logical plans after analysis [spark]

2025-01-30 Thread via GitHub
hvanhovell commented on code in PR #49584: URL: https://github.com/apache/spark/pull/49584#discussion_r1936120463 ## sql/connect/server/src/main/scala/org/apache/spark/sql/connect/service/SessionHolder.scala: ## @@ -440,46 +443,64 @@ case class SessionHolder(userId: String, sess