Serge Rielau created SPARK-39558:
------------------------------------

             Summary: Store error message  parameters as Map instead of Array
                 Key: SPARK-39558
                 URL: https://issues.apache.org/jira/browse/SPARK-39558
             Project: Spark
          Issue Type: Improvement
          Components: Spark Core
    Affects Versions: 3.3.1
            Reporter: Serge Rielau


Right now when we raise a SparkException we pass an array of argument which are 
assigned to the message parameters by position. This has several downsides:
1. It makes hit hard to later localize (or rework) the messages since that may 
shuffle position.
2. There could be an accidental mismatch when writing code which is not 
detected in QA.

3. Sometimes we want to use the same parameter multiple times in a message. 
Repeating it as an argument seems silly. 

All of these problems go away when we use a map aligning parameters and 
arguments. We already do this for CheckError.



--
This message was sent by Atlassian Jira
(v8.20.7#820007)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@spark.apache.org
For additional commands, e-mail: issues-h...@spark.apache.org

Reply via email to