[GitHub] [spark] mridulm commented on a diff in pull request #39226: [SPARK-41694][CORE] Add new config to clean up `spark.ui.store.path` directory when `SparkContext.stop()`

2022-12-30 Thread GitBox
mridulm commented on code in PR #39226: URL: https://github.com/apache/spark/pull/39226#discussion_r1059289062 ## core/src/main/scala/org/apache/spark/status/AppStatusStore.scala: ## @@ -733,6 +734,15 @@ private[spark] class AppStatusStore( def close(): Unit = { store.

[GitHub] [spark] mridulm commented on a diff in pull request #39226: [SPARK-41694][CORE] Add new config to clean up `spark.ui.store.path` directory when `SparkContext.stop()`

2022-12-30 Thread GitBox
mridulm commented on code in PR #39226: URL: https://github.com/apache/spark/pull/39226#discussion_r1059289062 ## core/src/main/scala/org/apache/spark/status/AppStatusStore.scala: ## @@ -733,6 +734,15 @@ private[spark] class AppStatusStore( def close(): Unit = { store.

[GitHub] [spark] mridulm commented on a diff in pull request #39226: [SPARK-41694][CORE] Add new config to clean up `spark.ui.store.path` directory when `SparkContext.stop()`

2022-12-30 Thread GitBox
mridulm commented on code in PR #39226: URL: https://github.com/apache/spark/pull/39226#discussion_r1059289062 ## core/src/main/scala/org/apache/spark/status/AppStatusStore.scala: ## @@ -733,6 +734,15 @@ private[spark] class AppStatusStore( def close(): Unit = { store.

[GitHub] [spark] mridulm commented on a diff in pull request #39226: [SPARK-41694][CORE] Add new config to clean up `spark.ui.store.path` directory when `SparkContext.stop()`

2022-12-28 Thread GitBox
mridulm commented on code in PR #39226: URL: https://github.com/apache/spark/pull/39226#discussion_r1058705480 ## core/src/main/scala/org/apache/spark/status/AppStatusStore.scala: ## @@ -733,6 +734,15 @@ private[spark] class AppStatusStore( def close(): Unit = { store.

[GitHub] [spark] mridulm commented on a diff in pull request #39226: [SPARK-41694][CORE] Add new config to clean up `spark.ui.store.path` directory when `SparkContext.stop()`

2022-12-27 Thread GitBox
mridulm commented on code in PR #39226: URL: https://github.com/apache/spark/pull/39226#discussion_r1057840697 ## core/src/main/scala/org/apache/spark/status/AppStatusStore.scala: ## @@ -733,6 +734,15 @@ private[spark] class AppStatusStore( def close(): Unit = { store.

[GitHub] [spark] mridulm commented on a diff in pull request #39226: [SPARK-41694][CORE] Add new config to clean up `spark.ui.store.path` directory when `SparkContext.stop()`

2022-12-27 Thread GitBox
mridulm commented on code in PR #39226: URL: https://github.com/apache/spark/pull/39226#discussion_r1057840697 ## core/src/main/scala/org/apache/spark/status/AppStatusStore.scala: ## @@ -733,6 +734,15 @@ private[spark] class AppStatusStore( def close(): Unit = { store.