samza-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From fredji97 <...@git.apache.org>
Subject [GitHub] samza pull request #392: SAMZA-1541 migrate config classes in samza-yarn to ...
Date Thu, 21 Dec 2017 23:53:33 GMT
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

----


---

Mime
View raw message