Re: [PR] [SPARK-44223][CORE] Remove LevelDB support [spark]

2025-06-01 Thread via GitHub
mridulm commented on PR #51027: URL: https://github.com/apache/spark/pull/51027#issuecomment-2929028378 @HeartSaVioR has raised a valid concern - what is the path to migration for existing users from level db to rocks db ? -- This is an automated message from the Apache Git Service. To re

Re: [PR] [SPARK-44223][CORE] Remove LevelDB support [spark]

2025-05-27 Thread via GitHub
Hisoka-X commented on code in PR #51027: URL: https://github.com/apache/spark/pull/51027#discussion_r2108870247 ## pom.xml: ## @@ -3575,7 +3564,7 @@ applesilicon - org.apache.spark.tags.ChromeUITest,org.apache.spark.tags.ExtendedLevelDBTest +

Re: [PR] [SPARK-44223][CORE] Remove LevelDB support [spark]

2025-05-27 Thread via GitHub
LuciferYang commented on code in PR #51027: URL: https://github.com/apache/spark/pull/51027#discussion_r2108696385 ## pom.xml: ## @@ -3575,7 +3564,7 @@ applesilicon - org.apache.spark.tags.ChromeUITest,org.apache.spark.tags.ExtendedLevelDBTest +

Re: [PR] [SPARK-44223][CORE] Remove LevelDB support [spark]

2025-05-27 Thread via GitHub
Hisoka-X commented on PR #51027: URL: https://github.com/apache/spark/pull/51027#issuecomment-2911644087 > To @Hisoka-X, perhaps we achieved a limited agreement in the previous pr, but I suggest that it would be better to initiate another relevant discussion on the dev mail list to reach a

Re: [PR] [SPARK-44223][CORE] Remove LevelDB support [spark]

2025-05-27 Thread via GitHub
LuciferYang commented on PR #51027: URL: https://github.com/apache/spark/pull/51027#issuecomment-2911570368 To @Hisoka-X, perhaps we achieved a limited agreement in the previous pr, but I suggest that it would be better to initiate another relevant discussion on the dev mail list to reach a

Re: [PR] [SPARK-44223][CORE] Remove LevelDB support [spark]

2025-05-27 Thread via GitHub
LuciferYang commented on PR #51027: URL: https://github.com/apache/spark/pull/51027#issuecomment-2911556333 also cc @mridulm -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment.

Re: [PR] [SPARK-44223][CORE] Remove LevelDB support [spark]

2025-05-27 Thread via GitHub
Hisoka-X commented on PR #51027: URL: https://github.com/apache/spark/pull/51027#issuecomment-2911550276 cc @LuciferYang @dongjoon-hyun -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specif

Re: [PR] [SPARK-44223][CORE] Remove LevelDB support [spark]

2025-05-27 Thread via GitHub
Hisoka-X commented on code in PR #43138: URL: https://github.com/apache/spark/pull/43138#discussion_r2108480314 ## core/src/main/scala/org/apache/spark/internal/config/package.scala: ## @@ -726,12 +726,12 @@ package object config { private[spark] val SHUFFLE_SERVICE_DB_BACKEN

Re: [PR] [SPARK-44223][CORE] Remove LevelDB support [spark]

2025-05-27 Thread via GitHub
LuciferYang commented on code in PR #43138: URL: https://github.com/apache/spark/pull/43138#discussion_r2108467286 ## core/src/main/scala/org/apache/spark/internal/config/package.scala: ## @@ -726,12 +726,12 @@ package object config { private[spark] val SHUFFLE_SERVICE_DB_BAC

Re: [PR] [SPARK-44223][CORE] Remove LevelDB support [spark]

2025-05-27 Thread via GitHub
Hisoka-X commented on code in PR #43138: URL: https://github.com/apache/spark/pull/43138#discussion_r2108448317 ## core/src/main/scala/org/apache/spark/internal/config/package.scala: ## @@ -726,12 +726,12 @@ package object config { private[spark] val SHUFFLE_SERVICE_DB_BACKEN

Re: [PR] [SPARK-44223][CORE] Remove LevelDB support [spark]

2025-05-25 Thread via GitHub
Hisoka-X commented on code in PR #43138: URL: https://github.com/apache/spark/pull/43138#discussion_r2106403777 ## core/src/main/scala/org/apache/spark/internal/config/package.scala: ## @@ -726,12 +726,12 @@ package object config { private[spark] val SHUFFLE_SERVICE_DB_BACKEN

Re: [PR] [SPARK-44223][CORE] Remove LevelDB support [spark]

2025-05-23 Thread via GitHub
LuciferYang commented on code in PR #43138: URL: https://github.com/apache/spark/pull/43138#discussion_r2105701489 ## core/src/main/scala/org/apache/spark/internal/config/package.scala: ## @@ -726,12 +726,12 @@ package object config { private[spark] val SHUFFLE_SERVICE_DB_BAC

Re: [PR] [SPARK-44223][CORE] Remove LevelDB support [spark]

2024-01-13 Thread via GitHub
github-actions[bot] closed pull request #43138: [SPARK-44223][CORE] Remove LevelDB support URL: https://github.com/apache/spark/pull/43138 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specifi

Re: [PR] [SPARK-44223][CORE] Remove LevelDB support [spark]

2024-01-12 Thread via GitHub
github-actions[bot] commented on PR #43138: URL: https://github.com/apache/spark/pull/43138#issuecomment-1890172267 We're closing this PR because it hasn't been updated in a while. This isn't a judgement on the merit of the PR in any way. It's just a way of keeping the PR queue manageable.

Re: [PR] [SPARK-44223][CORE] Remove LevelDB support [spark]

2023-10-03 Thread via GitHub
LuciferYang commented on PR #43138: URL: https://github.com/apache/spark/pull/43138#issuecomment-1746065245 @Hisoka-X Can you restart this job? I believe we can still remove Leveldb's support for HybridStore in Spark 4.0. -- This is an automated message from the Apache Git Service