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

Adam Antal commented on SUBMARINE-49:
-------------------------------------

Hi [~snemeth], sorry I didn't have the chance to look at it this week, but I am 
going to allocate some time next week.

> Add more test coverage to RunJobParameters
> ------------------------------------------
>
>                 Key: SUBMARINE-49
>                 URL: https://issues.apache.org/jira/browse/SUBMARINE-49
>             Project: Hadoop Submarine
>          Issue Type: Sub-task
>            Reporter: Szilard Nemeth
>            Assignee: Szilard Nemeth
>            Priority: Minor
>         Attachments: SUBMARINE-49.001.patch, SUBMARINE-49.002.patch, 
> SUBMARINE-49.003.patch
>
>
> There are some good tests in 
> {{org.apache.hadoop.yarn.submarine.client.cli.TestRunJobCliParsing}}, but 
> these are not testing all fields set by method 
> {{org.apache.hadoop.yarn.submarine.client.cli.param.RunJobParameters#updateParametersByParsedCommandline}}.
>  
> Some more extensive testing is needed in this area.
> As an added bonus, the code 
> {{org.apache.hadoop.yarn.submarine.client.cli.param.RunJobParameters#updateParametersByParsedCommandline}}
>  could be cleaned up a bit.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to