You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@spark.apache.org by "Hyukjin Kwon (Jira)" <ji...@apache.org> on 2022/04/20 02:49:00 UTC
[jira] [Commented] (SPARK-38955) from_csv can corrupt surrounding lines if a lineSep is in the data
[ https://issues.apache.org/jira/browse/SPARK-38955?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17524676#comment-17524676 ]
Hyukjin Kwon commented on SPARK-38955:
--------------------------------------
lineSep doesn't work with from_csv. We should probably ban this option within this expression.
> from_csv can corrupt surrounding lines if a lineSep is in the data
> ------------------------------------------------------------------
>
> Key: SPARK-38955
> URL: https://issues.apache.org/jira/browse/SPARK-38955
> Project: Spark
> Issue Type: Bug
> Components: SQL
> Affects Versions: 3.2.0
> Reporter: Robert Joseph Evans
> Priority: Major
>
> I don't know how critical this is. I was doing some general testing to understand {{from_csv}} and found that if I happen to have a {{lineSep}} in the input data and I noticed that the next row appears to be corrupted. {{multiLine}} does not appear to fix it. Because this is data corruption I am inclined to mark this as CRITICAL or BLOCKER, but it is an odd corner case so I m not going to set it myself.
> {code}
> Seq[String]("1,\n2,3,4,5","6,7,8,9,10", "11,12,13,14,15", null).toDF.select(col("value"), from_csv(col("value"), StructType(Seq(StructField("a", LongType), StructField("b", StringType))), Map[String,String]())).show()
> +--------------+---------------+
> | value|from_csv(value)|
> +--------------+---------------+
> | 1,\n2,3,4,5| {1, null}|
> | 6,7,8,9,10| {null, 8}|
> |11,12,13,14,15| {11, 12}|
> | null| null|
> +--------------+---------------+
> {code}
> {code}
> Seq[String]("1,:2,3,4,5","6,7,8,9,10", "11,12,13,14,15", null).toDF.select(col("value"), from_csv(col("value"), StructType(Seq(StructField("a", LongType), StructField("b", StringType))), Map[String,String]("lineSep" -> ":"))).show()
> +--------------+---------------+
> | value|from_csv(value)|
> +--------------+---------------+
> | 1,:2,3,4,5| {1, null}|
> | 6,7,8,9,10| {null, 8}|
> |11,12,13,14,15| {11, 12}|
> | null| null|
> +--------------+---------------+
> {code}
> {code}
> Seq[String]("1,\n2,3,4,5","6,7,8,9,10", "11,12,13,14,15", null).toDF.select(col("value"), from_csv(col("value"), StructType(Seq(StructField("a", LongType), StructField("b", StringType))), Map[String,String]("lineSep" -> ":"))).show()
> +--------------+---------------+
> | value|from_csv(value)|
> +--------------+---------------+
> | 1,\n2,3,4,5| {1, \n2}|
> | 6,7,8,9,10| {6, 7}|
> |11,12,13,14,15| {11, 12}|
> | null| null|
> +--------------+---------------+
> {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