Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/21185#discussion_r186772234
--- Diff: core/src/main/scala/org/apache/spark/executor/Executor.scala ---
@@ -229,6 +229,23 @@ private[spark] class Executor(
Github user squito closed the pull request at:
https://github.com/apache/spark/pull/21185
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21185#discussion_r186358733
--- Diff: core/src/main/scala/org/apache/spark/executor/Executor.scala ---
@@ -229,6 +229,23 @@ private[spark] class Executor(
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/21185#discussion_r185612118
--- Diff: core/src/main/scala/org/apache/spark/executor/Executor.scala ---
@@ -229,6 +229,23 @@ private[spark] class Executor(
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/21185#discussion_r184993529
--- Diff: core/src/main/scala/org/apache/spark/executor/Executor.scala ---
@@ -299,6 +316,9 @@ private[spark] class Executor(
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21185#discussion_r184813348
--- Diff: core/src/main/scala/org/apache/spark/executor/Executor.scala ---
@@ -299,6 +316,9 @@ private[spark] class Executor(
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21185#discussion_r184813243
--- Diff: core/src/main/scala/org/apache/spark/executor/Executor.scala ---
@@ -229,6 +229,23 @@ private[spark] class Executor(
GitHub user squito opened a pull request:
https://github.com/apache/spark/pull/21185
[SPARK-23894][CORE][SQL] Defensively clear ActiveSession in Executors
Because SparkSession.getActiveSession uses an InheritableThreadLocal,
the ThreadPool in executors might end up inheriting
