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