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

Srikanth Sundarrajan commented on FALCON-896:
---------------------------------------------

Thanks [~peeyushb], latest patch looks good, but there this does introduce 
checkstyle errors. Can you address them please?

{code}
[INFO] --- maven-checkstyle-plugin:2.9.1:check (checkstyle-check) @ 
falcon-hadoop-webapp ---
[INFO] Starting audit...
/projects/incubator-falcon/test-tools/hadoop-webapp/src/main/java/org/apache/falcon/listener/HadoopStartupListener.java:61:75:
 ',' is not followed by whitespace.
/projects/incubator-falcon/test-tools/hadoop-webapp/src/main/java/org/apache/falcon/listener/HadoopStartupListener.java:62:33:
 Array brackets at illegal position.
/projects/incubator-falcon/test-tools/hadoop-webapp/src/main/java/org/apache/falcon/listener/HadoopStartupListener.java:64:53:
 ',' is not followed by whitespace.
/projects/incubator-falcon/test-tools/hadoop-webapp/src/main/java/org/apache/falcon/listener/HadoopStartupListener.java:122:53:
 'shareLibPath' hides a field.
/projects/incubator-falcon/test-tools/hadoop-webapp/src/main/java/org/apache/falcon/listener/HadoopStartupListener.java:127:15:
 'if' is not followed by whitespace.
/projects/incubator-falcon/test-tools/hadoop-webapp/src/main/java/org/apache/falcon/listener/HadoopStartupListener.java:132:48:
 ',' is not followed by whitespace.
Audit done.

[INFO] There are 6 checkstyle errors.
{code}

> Upgrade oozie to 4.1.0 and make it the default for falcon
> ---------------------------------------------------------
>
>                 Key: FALCON-896
>                 URL: https://issues.apache.org/jira/browse/FALCON-896
>             Project: Falcon
>          Issue Type: Bug
>          Components: oozie
>    Affects Versions: 0.6
>            Reporter: Venkatesh Seetharam
>            Assignee: Peeyush Bishnoi
>         Attachments: FALCON-896.patch, FALCON-896.v2.patch
>
>




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

Reply via email to