[jira] [Comment Edited] (SPARK-39293) The accumulator of ArrayAggregate should copy the intermediate result if string, struct, array, or map

["Dongjoon Hyun (Jira)" <ji...@apache.org>]

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

Dongjoon Hyun edited comment on SPARK-39293 at 5/27/22 8:34 PM:
----------------------------------------------------------------

According to [~ueshin]'s email, I raised the priority as a blocker for Apache Spark 3.3.
This is not a regression, but a correctness issue.


was (Author: dongjoon):
According to [~ueshin]'s email, I raised the priority as a blocker for Apache Spark 3.3.

> The accumulator of ArrayAggregate should copy the intermediate result if string, struct, array, or map
> ------------------------------------------------------------------------------------------------------
>
>                 Key: SPARK-39293
>                 URL: https://issues.apache.org/jira/browse/SPARK-39293
>             Project: Spark
>          Issue Type: Bug
>          Components: SQL
>    Affects Versions: 3.0.3, 3.1.2, 3.2.1, 3.3.0
>            Reporter: Takuya Ueshin
>            Assignee: Takuya Ueshin
>            Priority: Blocker
>              Labels: correctness
>             Fix For: 3.1.3, 3.0.4, 3.2.2, 3.3.1
>
>
> The accumulator of ArrayAggregate should copy the intermediate result if string, struct, array, or map.
> {code:scala}
> import org.apache.spark.sql.functions._
> val reverse = udf((s: String) => s.reverse)
> val df = Seq(Array("abc", "def")).toDF("array")
> val testArray = df.withColumn(
>   "agg",
>   aggregate(
>     col("array"),
>     array().cast("array<string>"),
>     (acc, s) => concat(acc, array(reverse(s)))))
> aggArray.show(truncate=false)
> {code}
> should be:
> {code}
> +----------+----------+
> |array     |agg       |
> +----------+----------+
> |[abc, def]|[cba, fed]|
> +----------+----------+
> {code}
> but:
> {code}
> +----------+----------+
> |array     |agg       |
> +----------+----------+
> |[abc, def]|[fed, fed]|
> +----------+----------+
> {code}



--
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