[ 
https://issues.apache.org/jira/browse/FLINK-17023?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Caizhi Weng updated FLINK-17023:
--------------------------------
    Description: 
In FLINK-15727 {{BashJavaUtils}} now returns multiple lines of results to avoid 
using {{BashJavaUtils}} twice. But now the format checking in 
{{extractExecutionParams}} for the last line is incorrect. Instead of
{code:bash}
if ! [[ $execution_config =~ ^${EXECUTION_PREFIX}.* ]]; then
    echo "[ERROR] Unexpected result: $execution_config" 1>&2
    echo "[ERROR] The last line of the BashJavaUtils outputs is expected to be 
the execution result, following the prefix '${EXECUTION_PREFIX}'" 1>&2
    echo "$output" 1>&2
    exit 1
fi
{code}
It should be
{code:bash}
last_line=`echo "$execution_config" | tail -n 1`
if ! [[ "$last_line" =~ ^${EXECUTION_PREFIX}.* ]]; then
# ...
{code}

  was:
In FLINK-15727 extractExecutionParams now returns multiple lines of results to 
avoid using BashJavaUtils twice. But now the format checking for the last line 
is incorrect. Instead of
{code:bash}
if ! [[ $execution_config =~ ^${EXECUTION_PREFIX}.* ]]; then
    echo "[ERROR] Unexpected result: $execution_config" 1>&2
    echo "[ERROR] The last line of the BashJavaUtils outputs is expected to be 
the execution result, following the prefix '${EXECUTION_PREFIX}'" 1>&2
    echo "$output" 1>&2
    exit 1
fi
{code}
It should be
{code:bash}
last_line=`echo "$execution_config" | tail -n 1`
if ! [[ "$last_line" =~ ^${EXECUTION_PREFIX}.* ]]; then
# ...
{code}


> The format checking of extractExecutionParams in config.sh is incorrect
> -----------------------------------------------------------------------
>
>                 Key: FLINK-17023
>                 URL: https://issues.apache.org/jira/browse/FLINK-17023
>             Project: Flink
>          Issue Type: Bug
>          Components: Deployment / Scripts
>    Affects Versions: 1.11.0
>            Reporter: Caizhi Weng
>            Assignee: Caizhi Weng
>            Priority: Major
>              Labels: pull-request-available
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> In FLINK-15727 {{BashJavaUtils}} now returns multiple lines of results to 
> avoid using {{BashJavaUtils}} twice. But now the format checking in 
> {{extractExecutionParams}} for the last line is incorrect. Instead of
> {code:bash}
> if ! [[ $execution_config =~ ^${EXECUTION_PREFIX}.* ]]; then
>     echo "[ERROR] Unexpected result: $execution_config" 1>&2
>     echo "[ERROR] The last line of the BashJavaUtils outputs is expected to 
> be the execution result, following the prefix '${EXECUTION_PREFIX}'" 1>&2
>     echo "$output" 1>&2
>     exit 1
> fi
> {code}
> It should be
> {code:bash}
> last_line=`echo "$execution_config" | tail -n 1`
> if ! [[ "$last_line" =~ ^${EXECUTION_PREFIX}.* ]]; then
> # ...
> {code}



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

Reply via email to