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

ASF GitHub Bot commented on FLINK-2698:
---------------------------------------

Github user rmetzger commented on the pull request:

    https://github.com/apache/flink/pull/1142#issuecomment-141385220
  
    +1 to merge


> Add trailing newline to flink-conf.yaml
> ---------------------------------------
>
>                 Key: FLINK-2698
>                 URL: https://issues.apache.org/jira/browse/FLINK-2698
>             Project: Flink
>          Issue Type: Improvement
>    Affects Versions: master
>            Reporter: Greg Hogan
>            Assignee: Greg Hogan
>            Priority: Minor
>
> The distributed flink-conf.yaml does not contain a trailing newline. This 
> interferes with 
> [bdutil|https://github.com/GoogleCloudPlatform/bdutil/blob/master/extensions/flink/install_flink.sh#L64]
>  which appends extra/override configuration parameters with a heredoc.
> There are many other files without trailing newlines, but this looks to be 
> the only detrimental effect.
> {code}
> for i in $(find * -type f) ; do if diff /dev/null "$i" | tail -1 | grep '^\\ 
> No newline' > /dev/null; then  echo $i; fi; done
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to