falcon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Paul Isaychuk (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (FALCON-1011) Fix PrismProcessScheduleTest in falcon regression
Date Thu, 05 Feb 2015 14:31:34 GMT

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

Paul Isaychuk commented on FALCON-1011:
---------------------------------------

[~pragya.mittal] we always upload patches on review board(https://reviews.apache.org/dashboard/)
- and submit patches here in jira to see them in auto-generated reports:)
Generally it looks good except checkstyle issues:
{code}
/home/fertrist/work/apache-falcon/falcon-regression/merlin/src/test/java/org/apache/falcon/regression/prism/PrismProcessScheduleTest.java:44:8:
Unused import - java.util.logging.Logger.
/home/fertrist/work/apache-falcon/falcon-regression/merlin/src/test/java/org/apache/falcon/regression/prism/PrismProcessScheduleTest.java:331:71:
',' is not followed by whitespace.
/home/fertrist/work/apache-falcon/falcon-regression/merlin/src/test/java/org/apache/falcon/regression/prism/PrismProcessScheduleTest.java:339:
Line is longer than 120 characters.
{code}
So my (y) after review comments be addressed.

> Fix PrismProcessScheduleTest in falcon regression 
> --------------------------------------------------
>
>                 Key: FALCON-1011
>                 URL: https://issues.apache.org/jira/browse/FALCON-1011
>             Project: Falcon
>          Issue Type: Bug
>          Components: merlin
>    Affects Versions: 0.7
>            Reporter: Pragya Mittal
>            Assignee: Pragya Mittal
>         Attachments: FALCON-1011.patch
>
>
> Fixing few faliling test in PrismProcessScheduleTest.



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

Mime
View raw message