[ https://issues.apache.org/jira/browse/SAMZA-1541?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16300741#comment-16300741 ]
ASF GitHub Bot commented on SAMZA-1541: --------------------------------------- GitHub user fredji97 opened a pull request: https://github.com/apache/samza/pull/392 SAMZA-1541 migrate config classes in samza-yarn to use composition over inherence ./gradlew clean check successfully. The reason for doing this is described in https://issues.apache.org/jira/browse/SAMZA-1160. I am going to migrate for each directory and this PR is for samza-yarn. Besides the refactoring, two config class files (LocalizerResourceConfig.java and FileSystemImplConfig.java) are also moved from org.apache.samza.job.yarn to org.apache.samza.config. You can merge this pull request into a Git repository by running: $ git pull https://github.com/fredji97/samza samzaYarnConfig Alternatively you can review and apply these changes as the patch at: https://github.com/apache/samza/pull/392.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #392 ---- commit b3a64c9384c54a206c60505a04c3eed41a6a562b Author: Fred Ji <haifeng.ji@...> Date: 2017-12-21T23:49:04Z SAMZA-1541 migrate config classes in samza-yarn to use composition over inherence ---- > migrate config classes in samza-yarn to use composition over inherence > ---------------------------------------------------------------------- > > Key: SAMZA-1541 > URL: https://issues.apache.org/jira/browse/SAMZA-1541 > Project: Samza > Issue Type: Improvement > Reporter: Fred Ji > Assignee: Fred Ji > Fix For: 0.15.0 > > > This is one of the child tasks from > https://issues.apache.org/jira/browse/SAMZA-1160 -- This message was sent by Atlassian JIRA (v6.4.14#64029)