[
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)
