You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spark.apache.org by ma...@apache.org on 2024/08/14 11:50:43 UTC

(spark) branch master updated: [SPARK-49222][SQL] Clear useless methods in QueryCompilationErrors

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

maxgekk 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 02795a36171c [SPARK-49222][SQL] Clear useless methods in QueryCompilationErrors
02795a36171c is described below

commit 02795a36171cac51b1beaf68c42e8dfe3e6f0903
Author: xuyu <11...@vivo.com>
AuthorDate: Wed Aug 14 13:50:31 2024 +0200

    [SPARK-49222][SQL] Clear useless methods in QueryCompilationErrors
    
    ### What changes were proposed in this pull request?
    
    Clear useless methods in QueryCompilationErrors
    
    ### Why are the changes needed?
    
    Some unused methods in QueryCompilationErrors need to clear
    
    ### Does this PR introduce _any_ user-facing change?
    
    No
    
    ### How was this patch tested?
    
    Not need
    
    ### Was this patch authored or co-authored using generative AI tooling?
    
    No
    
    Closes #47740 from xuzifu666/clean_QueryCompilationErrors.
    
    Authored-by: xuyu <11...@vivo.com>
    Signed-off-by: Max Gekk <ma...@gmail.com>
---
 common/utils/src/main/resources/error/error-conditions.json |  6 ------
 .../apache/spark/sql/errors/QueryCompilationErrors.scala    | 13 -------------
 2 files changed, 19 deletions(-)

diff --git a/common/utils/src/main/resources/error/error-conditions.json b/common/utils/src/main/resources/error/error-conditions.json
index 30e9f0a246e3..957ffcfd81ed 100644
--- a/common/utils/src/main/resources/error/error-conditions.json
+++ b/common/utils/src/main/resources/error/error-conditions.json
@@ -388,12 +388,6 @@
     ],
     "sqlState" : "58030"
   },
-  "CANNOT_SAVE_VARIANT" : {
-    "message" : [
-      "Cannot save variant data type into external storage."
-    ],
-    "sqlState" : "0A000"
-  },
   "CANNOT_UPDATE_FIELD" : {
     "message" : [
       "Cannot update <table> field <fieldName> type:"
diff --git a/sql/catalyst/src/main/scala/org/apache/spark/sql/errors/QueryCompilationErrors.scala b/sql/catalyst/src/main/scala/org/apache/spark/sql/errors/QueryCompilationErrors.scala
index c422b32c5ec2..5dc7a5a7821d 100644
--- a/sql/catalyst/src/main/scala/org/apache/spark/sql/errors/QueryCompilationErrors.scala
+++ b/sql/catalyst/src/main/scala/org/apache/spark/sql/errors/QueryCompilationErrors.scala
@@ -203,13 +203,6 @@ private[sql] object QueryCompilationErrors extends QueryErrorsBase with Compilat
     invalidParameter("DOUBLE", "ewm", "alpha", invalidValue)
   }
 
-  def invalidStringParameter(
-      functionName: String,
-      parameter: String,
-      invalidValue: Expression): Throwable = {
-    invalidParameter("STRING", functionName, parameter, invalidValue)
-  }
-
   def invalidParameter(
       subClass: String,
       functionName: String,
@@ -1679,12 +1672,6 @@ private[sql] object QueryCompilationErrors extends QueryErrorsBase with Compilat
       messageParameters = Map.empty)
   }
 
-  def cannotSaveVariantIntoExternalStorageError(): Throwable = {
-    new AnalysisException(
-      errorClass = "CANNOT_SAVE_VARIANT",
-      messageParameters = Map.empty)
-  }
-
   def cannotResolveAttributeError(name: String, outputStr: String): Throwable = {
     new AnalysisException(
       errorClass = "_LEGACY_ERROR_TEMP_1137",


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