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

Peter Cseh commented on OOZIE-2813:
-----------------------------------

The 
[raw-patch-analisys|https://github.com/apache/oozie/blob/master/bin/test-patch-05-patch-raw-analysis#L85]
 step in our pre-commit hook checks for tabs and trailing whitespaces in the 
patches. They should be able to catch these in future commits.
Btw, there was no build for this patch from HadoopQA. It shouldn't break 
anything, but I wonder why it's missing. 


> Remove tabs and trailing whitespaces from oozie-defaul.xml
> ----------------------------------------------------------
>
>                 Key: OOZIE-2813
>                 URL: https://issues.apache.org/jira/browse/OOZIE-2813
>             Project: Oozie
>          Issue Type: Bug
>            Reporter: Peter Cseh
>            Assignee: Peter Cseh
>              Labels: newbie
>         Attachments: OOZIE-2813-001.patch
>
>
> oozie-default.xml has a buch of tab characters and trailing whitespaces in it.
> >> [9] 09:41 : oozie (master) :: grep "\t" 
> >> core/src/main/resources/oozie-default.xml |wc -l
>       75
> >> [10] 09:41 : oozie (master) :: grep " $" 
> >> core/src/main/resources/oozie-default.xml |wc -l
>       17
> Trailing whitespaces should be removed, while tabs have to replaced with 
> spaces.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to