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

Jark Wu commented on FLINK-15223:
---------------------------------

> if user really want to "\n" instead of line break
Then the user should use '\\n' in text. 

> the text of tEnv.sqlUpdate should not be unescaped
I agree the whole text shouldn't be unescaped. What I suggested is only to 
unescape the property values in {{SqlToOperationConverter#convertCreateTable}}.

> please do unescape by SQL platform user themselves, we do not need to do 
> these unnecessary things. 
This is the same to fix in SQL CLI side, as I said before, it is tricky and 
error-prone to unescape the whole SQL. 




> Csv connector should unescape delimiter parameter characters
> ------------------------------------------------------------
>
>                 Key: FLINK-15223
>                 URL: https://issues.apache.org/jira/browse/FLINK-15223
>             Project: Flink
>          Issue Type: Bug
>          Components: Formats (JSON, Avro, Parquet, ORC, SequenceFile)
>            Reporter: Jark Wu
>            Assignee: Jingsong Lee
>            Priority: Blocker
>             Fix For: 1.10.0
>
>
> As described in documentation[1], a csv format can use 
> {{'format.line-delimiter' = '\n'}} to specify line delimiter. However, the 
> property value is parsed into two characters "\n" , this result to reading 
> failed. There is no workaround for now, unless fix it. The delimiter should 
> be unescaped, e.g. using {{StringEscapeUtils.unescapeJava}}. Note that both 
> old csv and new csv have the same problem, and both {{field-delimiter}} and 
> {{line-delimiter}}.
> [1]: 
> https://ci.apache.org/projects/flink/flink-docs-master/dev/table/connect.html#old-csv-format



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

Reply via email to