[jira] [Commented] (SPARK-36210) Preserve column insertion order in Dataset.withColumns

2021-07-19 Thread Apache Spark (Jira)


[ 
https://issues.apache.org/jira/browse/SPARK-36210?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17383415#comment-17383415
 ] 

Apache Spark commented on SPARK-36210:
--

User 'koertkuipers' has created a pull request for this issue:
https://github.com/apache/spark/pull/33423

> Preserve column insertion order in Dataset.withColumns
> --
>
> Key: SPARK-36210
> URL: https://issues.apache.org/jira/browse/SPARK-36210
> Project: Spark
>  Issue Type: Bug
>  Components: SQL
>Affects Versions: 3.1.2
>Reporter: koert kuipers
>Priority: Minor
>
> Dataset.withColumns uses a Map (columnMap) to store the mapping of column 
> name to column. however this loses the order of the columns. also none of the 
> operations used on the Map (find and filter) benefits from the map's lookup 
> features. so it seems simpler to use a Seq instead, which also preserves the 
> insertion order.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

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



[jira] [Commented] (SPARK-36210) Preserve column insertion order in Dataset.withColumns

2021-07-19 Thread Apache Spark (Jira)


[ 
https://issues.apache.org/jira/browse/SPARK-36210?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17383414#comment-17383414
 ] 

Apache Spark commented on SPARK-36210:
--

User 'koertkuipers' has created a pull request for this issue:
https://github.com/apache/spark/pull/33423

> Preserve column insertion order in Dataset.withColumns
> --
>
> Key: SPARK-36210
> URL: https://issues.apache.org/jira/browse/SPARK-36210
> Project: Spark
>  Issue Type: Bug
>  Components: SQL
>Affects Versions: 3.1.2
>Reporter: koert kuipers
>Priority: Minor
>
> Dataset.withColumns uses a Map (columnMap) to store the mapping of column 
> name to column. however this loses the order of the columns. also none of the 
> operations used on the Map (find and filter) benefits from the map's lookup 
> features. so it seems simpler to use a Seq instead, which also preserves the 
> insertion order.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

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