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

Peter Cseh commented on OOZIE-2918:
-----------------------------------

Thanks for working on this!
# TestLauncherMapper is testing the LauncherAMUtils for the most part, please 
rename it
# TestLauncher was deleted. I think most of the tests are not needed because of 
deletion and replacement is tested elsewhere, but can you double-check that?
# Please look into removing 
OOZieLauncherInputFormat/OOZieLauncherOutputFormat/OOZieLauncherOutputCommitter 
classes.

> Delete LauncherMapper and its test
> ----------------------------------
>
>                 Key: OOZIE-2918
>                 URL: https://issues.apache.org/jira/browse/OOZIE-2918
>             Project: Oozie
>          Issue Type: Sub-task
>            Reporter: Peter Cseh
>            Assignee: Attila Sasvari
>         Attachments: OOZIE-2918-01.patch
>
>
> While LauncherMapper itself is not used anymore it contains a bunch of static 
> methods that are referenced from different places in the code.
> We should refactor out code and related test cases to where they belong.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to