You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@spark.apache.org by "Serge Rielau (Jira)" <ji...@apache.org> on 2022/06/22 18:21:00 UTC

[jira] [Created] (SPARK-39558) Store error message parameters as Map instead of Array

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-unsubscribe@spark.apache.org
For additional commands, e-mail: issues-help@spark.apache.org