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