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

Billie Rinaldi commented on SLIDER-1107:
----------------------------------------

bq. Actually for some commands the --help does not work unless it explicitly 
calls actionHelp. If you run it local in your mac say, it does not work for the 
resources cmd, and throws BindException for not being able to connect to RM. It 
had to be done explicitly for flex, package and list commands.

I guess that only happens on a misconfigured Slider installation; I ran slider 
resource --help on mine and it worked. Seems reasonable to add the shortcut to 
actionHelp, though.

I believe I have addressed all your comments in my latest commit d3923d8. 
Please take a look and let me know if I missed anything, or if you notice any 
new issues. I also found a minor test failure that I'd missed before, so I 
fixed that as well. Unit tests are passing, and I am now trying the funtests 
with this patch.

> Generate app configuration files in AM
> --------------------------------------
>
>                 Key: SLIDER-1107
>                 URL: https://issues.apache.org/jira/browse/SLIDER-1107
>             Project: Slider
>          Issue Type: Bug
>            Reporter: Billie Rinaldi
>            Assignee: Billie Rinaldi
>             Fix For: Slider 1.0.0
>
>
> Currently, each container generates its own application configuration files.  
> Instead, we could do this in the AM and have YARN localize the configuration 
> files.  Having some basic config generation in the AM may allow us to 
> simplify the config generation code in the app packages.  Also, it would be 
> much better in the case of Docker containers, where we would prefer not to 
> have to execute our own code inside the container.



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

Reply via email to