You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spark.apache.org by ya...@apache.org on 2023/06/29 03:50:01 UTC

[spark] branch master updated: [SPARK-43265][CORE][FOLLOW-UP] Move Error framework to a common utils module

This is an automated email from the ASF dual-hosted git repository.

yao pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git


The following commit(s) were added to refs/heads/master by this push:
     new a1592fa7ff2 [SPARK-43265][CORE][FOLLOW-UP] Move Error framework to a common utils module
a1592fa7ff2 is described below

commit a1592fa7ff2674f6be956f86a61e8da3554601cf
Author: Rui Wang <ru...@databricks.com>
AuthorDate: Thu Jun 29 11:49:46 2023 +0800

    [SPARK-43265][CORE][FOLLOW-UP] Move Error framework to a common utils module
    
    ### What changes were proposed in this pull request?
    
    We can also move `error-classes.json` to `common-utils`.
    
    ### Why are the changes needed?
    
    So Scala client can re-use the file but do not need to depend on Spark core.
    
    ### Does this PR introduce _any_ user-facing change?
    
    No
    
    ### How was this patch tested?
    
    Existing tests
    
    Closes #41774 from amaliujia/move_error_file.
    
    Authored-by: Rui Wang <ru...@databricks.com>
    Signed-off-by: Kent Yao <ya...@apache.org>
---
 {core => common/utils}/src/main/resources/error/README.md          | 0
 {core => common/utils}/src/main/resources/error/error-classes.json | 0
 core/src/test/scala/org/apache/spark/SparkThrowableSuite.scala     | 2 +-
 3 files changed, 1 insertion(+), 1 deletion(-)

diff --git a/core/src/main/resources/error/README.md b/common/utils/src/main/resources/error/README.md
similarity index 100%
rename from core/src/main/resources/error/README.md
rename to common/utils/src/main/resources/error/README.md
diff --git a/core/src/main/resources/error/error-classes.json b/common/utils/src/main/resources/error/error-classes.json
similarity index 100%
rename from core/src/main/resources/error/error-classes.json
rename to common/utils/src/main/resources/error/error-classes.json
diff --git a/core/src/test/scala/org/apache/spark/SparkThrowableSuite.scala b/core/src/test/scala/org/apache/spark/SparkThrowableSuite.scala
index e9554da082a..96c4e3b8ab7 100644
--- a/core/src/test/scala/org/apache/spark/SparkThrowableSuite.scala
+++ b/core/src/test/scala/org/apache/spark/SparkThrowableSuite.scala
@@ -47,7 +47,7 @@ class SparkThrowableSuite extends SparkFunSuite {
    }}}
    */
   private val errorJsonFilePath = getWorkspaceFilePath(
-    "core", "src", "main", "resources", "error", "error-classes.json")
+    "common", "utils", "src", "main", "resources", "error", "error-classes.json")
 
   private val errorReader = new ErrorClassesJsonReader(Seq(errorJsonFilePath.toUri.toURL))
 


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@spark.apache.org
For additional commands, e-mail: commits-help@spark.apache.org