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

Martin Serrano edited comment on TWILL-217 at 2/24/17 6:09 PM:
---------------------------------------------------------------

Yeah, I will do what you suggest.  I considered that but it seemed a little 
hacky so I was trying to think of other things.  

Regarding guava, I see your point, and there are things in it that are no 
longer needed now that java has them (functions, predicates, etc).


was (Author: mserrano):
Yeah, I can do what you suggest.  I considered that but it seemed a little 
hacky so I was trying to think of other things.  

Regarding guava, I see your point, and there are things in it that are no 
longer needed now that java has them (functions, predicates, etc).

> AppMaster launcher should include eventHandler dependencies and nothing else 
> from application
> ---------------------------------------------------------------------------------------------
>
>                 Key: TWILL-217
>                 URL: https://issues.apache.org/jira/browse/TWILL-217
>             Project: Apache Twill
>          Issue Type: Improvement
>          Components: yarn
>    Affects Versions: 0.9.0
>            Reporter: Martin Serrano
>            Assignee: Martin Serrano
>
> Currently the launcher for the appmaster includes the application.jar 
> libraries.  This is to support user code that adds an EventHandler.  The 
> application may have many dependencies and including them in the appmaster 
> classpath can lead to otherwise inaddressable incompatibilities.
> In my case, something in my application's large dependency graph was 
> interfering with the Kafka server operation.  I was not able to determine 
> what it was but tweaking the appmaster loader to not include my application 
> jars fixed the issue.
> Instead the bundler that creates the twill.jar should include the 
> EventHandler extension (if any) as an explicit dependency.  In this way, only 
> the jars needed to support the event handler will be on the twill classpath.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to