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

Yang Wang commented on FLINK-15053:
-----------------------------------

All the value of dynamic properties will be put into a pair of single quotes. 
If the value contains single quote, will be replaced with <SINGLE_QUOTE>. When 
loading the dynamic properties, it will be restored to single quote. I do not 
suggest to use \' to escape, since it will not work for start command of bash.
{code:java}
// start command
bash -c "$JAVA_HOME/bin/java -Xmx424m ... ... 
org.apache.flink.yarn.entrypoint.YarnJobClusterEntrypoint -Dkey1='\'value' 
-Dkey2='20 s' -Dkey3='value3'"

bash -c "$JAVA_HOME/bin/java -Xmx424m ... ... 
org.apache.flink.yarn.entrypoint.YarnJobClusterEntrypoint 
-Dkey1='<SINGLE_QUOTE>value' -Dkey2='20 s' -Dkey3='value3'"{code}
[~gjy] What do you think?

> Configurations with values contains space may cause TM failures on Yarn
> -----------------------------------------------------------------------
>
>                 Key: FLINK-15053
>                 URL: https://issues.apache.org/jira/browse/FLINK-15053
>             Project: Flink
>          Issue Type: Bug
>          Components: Deployment / YARN, Runtime / Coordination
>    Affects Versions: 1.10.0
>            Reporter: Xintong Song
>            Priority: Major
>             Fix For: 1.10.0
>
>
> Currently on Yarn setups, we are passing task executor specific 
> configurations through dynamic properties in the starting command (see 
> FLINK-13184).
> If the value of configuration contains space, the dynamic properties may not 
> be correctly parsed, which could cause task executor failures. On occurrence 
> can be found in FLINK-15047.
> It would be good to allow spaces when passing dynamic properties. E.g., 
> surrounding the values with double quotation marks, or escaping special 
> characters.
> cc [~fly_in_gis]



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

Reply via email to