[GitHub] samza pull request #392: SAMZA-1541 migrate config classes in samza-yarn to ...
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 Date: 2017-12-21T23:49:04Z SAMZA-1541 migrate config classes in samza-yarn to use composition over inherence ---
Re: [RESULT][VOTE] Apache Samza 0.14.0 RC3
Thank you everyone for the effort, and to Xinyu for shepherding Samza 0.14! This includes an exciting line-up of features including - SQL support, Kinesis integration and EventHub support. On Thu, Dec 21, 2017 at 11:03 AM, xinyu liu wrote: > The vote of 0.14.0 RC3 has been more than 72 hours and we got +1 (binding) > x 3 and +1 (non-binding) x 3. > > Samza 0.14.0 officially passed the VOTE! > > Thanks! > Xinyu > -- Jagadish V, Graduate Student, Department of Computer Science, Stanford University
[GitHub] samza pull request #388: SAMZA-1537: StreamAppender can deadlock due to lock...
Github user asfgit closed the pull request at: https://github.com/apache/samza/pull/388 ---
[RESULT][VOTE] Apache Samza 0.14.0 RC3
The vote of 0.14.0 RC3 has been more than 72 hours and we got +1 (binding) x 3 and +1 (non-binding) x 3. Samza 0.14.0 officially passed the VOTE! Thanks! Xinyu
Re: [VOTE] Apache Samza 0.14.0 RC3
+1 Run check-all.sh successfully on both OSX and linux. Thanks, Xinyu On Thu, Dec 21, 2017 at 10:51 AM, Daniel Nishimura wrote: > +1 > Verified signatures and built on Ubuntu 14.04 and macOS 10.12 with JDK > 1.8.0_121 > Note: The build is currently works with Gradle 2.8. The following error > happens on higher versions of Gradle: “bad option: '-feature > -language:implicitConversions -language:reflectiveCalls'” > There’s a PR to address this in a future release: > https://github.com/apache/samza/pull/326 > > On 12/21/17, 8:10 AM, "Jagadish Venkatraman" > wrote: > > LGTM. > > I verified signatures and ran all tests on OsX. Additionally, > *check-all.sh* > succeeded. > > +1 (binding) > > > > > On Thu, Dec 21, 2017 at 1:07 AM, Yi Pan wrote: > > > +1 binding > > > > Verified git tag and source signatures > > > > Ran check-all.sh on OSX > > Ran integration tests from OSX > > > > Thanks for push it forward! > > > > -Yi > > > > On Wed, Dec 20, 2017 at 3:19 PM, Jake Maes > wrote: > > > > > +1 binding > > > > > > Verified git tag `release-0.14.0-rc3` > > > Verified source signature > > > > > > Ran check-all.sh on both Linux and OSX > > > Ran integration tests from OSX > > > > > > LGTM > > > > > > > > > On Tue, Dec 19, 2017 at 10:00 AM, Boris S > wrote: > > > > > > > Verified signature. > > > > Ran unit and integration tests. > > > > As usual had to force python to 2.7 for integration tests to run > on > > > Linux. > > > > > > > > +1 > > > > > > > > On Mon, Dec 18, 2017 at 11:01 AM, xinyu liu < > xinyuliu...@gmail.com> > > > wrote: > > > > > > > > > Correction: This is a call for a vote on a release of Apache > Samza > > > > > *0.14.0*. > > > > > > > > > > On Mon, Dec 18, 2017 at 10:57 AM, xinyu liu < > xinyuliu...@gmail.com> > > > > wrote: > > > > > > > > > > > This is a call for a vote on a release of Apache Samza > 0.13.1. > > Thanks > > > > to > > > > > everyone > > > > > > who has contributed to this release. > > > > > > > > > > > > The release candidate can be downloaded from here: > > > > > > http://home.apache.org/~xinyu/samza-0.14.0-rc3/ > > > > > > > > > > > > The release candidate is signed with pgp key C31D7061, which > can be > > > > > found on > > > > > > keyservers: > > > > > > http://pgp.mit.edu/pks/lookup?op=get&search= > 0x35964389C31D7061 > > > > > > > > > > > > The git tag is release-0.13.1-rc0 and signed with the same > pgp key: > > > > > > https://git-wip-us.apache.org/repos/asf?p=samza.git;a=tag;h= > > > > > > refs/tags/release-0.14.0-rc3 > > > > > > > > > > > > Test binaries have been published to Maven's staging > repository, > > and > > > > are > > > > > available > > > > > > here: > > > > > > https://repository.apache.org/content/repositories/ > > > orgapachesamza-1036 > > > > > > > > > > > > 61 issues have been resolved as part of this release > > > > > > https://issues.apache.org/jira/browse/SAMZA-1519?jql= > > > > > > project%20%3D%20SAMZA%20AND%20fixVersion%20%3D%200.14.0% > > > > > > 20AND%20status%20%3D%20Resolved > > > > > > > > > > > > The vote will be open for 72 hours (ending at 11:00 AM > Thursday, > > > > > > 12/21/2017). > > > > > > > > > > > > Please download the release candidate, check the > hashes/signature, > > > > build > > > > > > it and test it, and then please vote: > > > > > > > > > > > > [ ] +1 approve > > > > > > > > > > > > [ ] +0 no opinion > > > > > > > > > > > > [ ] -1 disapprove (and reason why) > > > > > > > > > > > > Thanks, > > > > > > Xinyu > > > > > > > > > > > > > > > > > > > > > > > > -- > Jagadish V, > Graduate Student, > Department of Computer Science, > Stanford University > > >
Re: [VOTE] Apache Samza 0.14.0 RC3
+1 Verified signatures and built on Ubuntu 14.04 and macOS 10.12 with JDK 1.8.0_121 Note: The build is currently works with Gradle 2.8. The following error happens on higher versions of Gradle: “bad option: '-feature -language:implicitConversions -language:reflectiveCalls'” There’s a PR to address this in a future release: https://github.com/apache/samza/pull/326 On 12/21/17, 8:10 AM, "Jagadish Venkatraman" wrote: LGTM. I verified signatures and ran all tests on OsX. Additionally, *check-all.sh* succeeded. +1 (binding) On Thu, Dec 21, 2017 at 1:07 AM, Yi Pan wrote: > +1 binding > > Verified git tag and source signatures > > Ran check-all.sh on OSX > Ran integration tests from OSX > > Thanks for push it forward! > > -Yi > > On Wed, Dec 20, 2017 at 3:19 PM, Jake Maes wrote: > > > +1 binding > > > > Verified git tag `release-0.14.0-rc3` > > Verified source signature > > > > Ran check-all.sh on both Linux and OSX > > Ran integration tests from OSX > > > > LGTM > > > > > > On Tue, Dec 19, 2017 at 10:00 AM, Boris S wrote: > > > > > Verified signature. > > > Ran unit and integration tests. > > > As usual had to force python to 2.7 for integration tests to run on > > Linux. > > > > > > +1 > > > > > > On Mon, Dec 18, 2017 at 11:01 AM, xinyu liu > > wrote: > > > > > > > Correction: This is a call for a vote on a release of Apache Samza > > > > *0.14.0*. > > > > > > > > On Mon, Dec 18, 2017 at 10:57 AM, xinyu liu > > > wrote: > > > > > > > > > This is a call for a vote on a release of Apache Samza 0.13.1. > Thanks > > > to > > > > everyone > > > > > who has contributed to this release. > > > > > > > > > > The release candidate can be downloaded from here: > > > > > http://home.apache.org/~xinyu/samza-0.14.0-rc3/ > > > > > > > > > > The release candidate is signed with pgp key C31D7061, which can be > > > > found on > > > > > keyservers: > > > > > http://pgp.mit.edu/pks/lookup?op=get&search=0x35964389C31D7061 > > > > > > > > > > The git tag is release-0.13.1-rc0 and signed with the same pgp key: > > > > > https://git-wip-us.apache.org/repos/asf?p=samza.git;a=tag;h= > > > > > refs/tags/release-0.14.0-rc3 > > > > > > > > > > Test binaries have been published to Maven's staging repository, > and > > > are > > > > available > > > > > here: > > > > > https://repository.apache.org/content/repositories/ > > orgapachesamza-1036 > > > > > > > > > > 61 issues have been resolved as part of this release > > > > > https://issues.apache.org/jira/browse/SAMZA-1519?jql= > > > > > project%20%3D%20SAMZA%20AND%20fixVersion%20%3D%200.14.0% > > > > > 20AND%20status%20%3D%20Resolved > > > > > > > > > > The vote will be open for 72 hours (ending at 11:00 AM Thursday, > > > > > 12/21/2017). > > > > > > > > > > Please download the release candidate, check the hashes/signature, > > > build > > > > > it and test it, and then please vote: > > > > > > > > > > [ ] +1 approve > > > > > > > > > > [ ] +0 no opinion > > > > > > > > > > [ ] -1 disapprove (and reason why) > > > > > > > > > > Thanks, > > > > > Xinyu > > > > > > > > > > > > > > > -- Jagadish V, Graduate Student, Department of Computer Science, Stanford University
[GitHub] samza pull request #391: Merge script improvement - use colons instead of se...
Github user asfgit closed the pull request at: https://github.com/apache/samza/pull/391 ---
Re: [VOTE] Apache Samza 0.14.0 RC3
LGTM. I verified signatures and ran all tests on OsX. Additionally, *check-all.sh* succeeded. +1 (binding) On Thu, Dec 21, 2017 at 1:07 AM, Yi Pan wrote: > +1 binding > > Verified git tag and source signatures > > Ran check-all.sh on OSX > Ran integration tests from OSX > > Thanks for push it forward! > > -Yi > > On Wed, Dec 20, 2017 at 3:19 PM, Jake Maes wrote: > > > +1 binding > > > > Verified git tag `release-0.14.0-rc3` > > Verified source signature > > > > Ran check-all.sh on both Linux and OSX > > Ran integration tests from OSX > > > > LGTM > > > > > > On Tue, Dec 19, 2017 at 10:00 AM, Boris S wrote: > > > > > Verified signature. > > > Ran unit and integration tests. > > > As usual had to force python to 2.7 for integration tests to run on > > Linux. > > > > > > +1 > > > > > > On Mon, Dec 18, 2017 at 11:01 AM, xinyu liu > > wrote: > > > > > > > Correction: This is a call for a vote on a release of Apache Samza > > > > *0.14.0*. > > > > > > > > On Mon, Dec 18, 2017 at 10:57 AM, xinyu liu > > > wrote: > > > > > > > > > This is a call for a vote on a release of Apache Samza 0.13.1. > Thanks > > > to > > > > everyone > > > > > who has contributed to this release. > > > > > > > > > > The release candidate can be downloaded from here: > > > > > http://home.apache.org/~xinyu/samza-0.14.0-rc3/ > > > > > > > > > > The release candidate is signed with pgp key C31D7061, which can be > > > > found on > > > > > keyservers: > > > > > http://pgp.mit.edu/pks/lookup?op=get&search=0x35964389C31D7061 > > > > > > > > > > The git tag is release-0.13.1-rc0 and signed with the same pgp key: > > > > > https://git-wip-us.apache.org/repos/asf?p=samza.git;a=tag;h= > > > > > refs/tags/release-0.14.0-rc3 > > > > > > > > > > Test binaries have been published to Maven's staging repository, > and > > > are > > > > available > > > > > here: > > > > > https://repository.apache.org/content/repositories/ > > orgapachesamza-1036 > > > > > > > > > > 61 issues have been resolved as part of this release > > > > > https://issues.apache.org/jira/browse/SAMZA-1519?jql= > > > > > project%20%3D%20SAMZA%20AND%20fixVersion%20%3D%200.14.0% > > > > > 20AND%20status%20%3D%20Resolved > > > > > > > > > > The vote will be open for 72 hours (ending at 11:00 AM Thursday, > > > > > 12/21/2017). > > > > > > > > > > Please download the release candidate, check the hashes/signature, > > > build > > > > > it and test it, and then please vote: > > > > > > > > > > [ ] +1 approve > > > > > > > > > > [ ] +0 no opinion > > > > > > > > > > [ ] -1 disapprove (and reason why) > > > > > > > > > > Thanks, > > > > > Xinyu > > > > > > > > > > > > > > > -- Jagadish V, Graduate Student, Department of Computer Science, Stanford University
Re: [VOTE] Apache Samza 0.14.0 RC3
+1 binding Verified git tag and source signatures Ran check-all.sh on OSX Ran integration tests from OSX Thanks for push it forward! -Yi On Wed, Dec 20, 2017 at 3:19 PM, Jake Maes wrote: > +1 binding > > Verified git tag `release-0.14.0-rc3` > Verified source signature > > Ran check-all.sh on both Linux and OSX > Ran integration tests from OSX > > LGTM > > > On Tue, Dec 19, 2017 at 10:00 AM, Boris S wrote: > > > Verified signature. > > Ran unit and integration tests. > > As usual had to force python to 2.7 for integration tests to run on > Linux. > > > > +1 > > > > On Mon, Dec 18, 2017 at 11:01 AM, xinyu liu > wrote: > > > > > Correction: This is a call for a vote on a release of Apache Samza > > > *0.14.0*. > > > > > > On Mon, Dec 18, 2017 at 10:57 AM, xinyu liu > > wrote: > > > > > > > This is a call for a vote on a release of Apache Samza 0.13.1. Thanks > > to > > > everyone > > > > who has contributed to this release. > > > > > > > > The release candidate can be downloaded from here: > > > > http://home.apache.org/~xinyu/samza-0.14.0-rc3/ > > > > > > > > The release candidate is signed with pgp key C31D7061, which can be > > > found on > > > > keyservers: > > > > http://pgp.mit.edu/pks/lookup?op=get&search=0x35964389C31D7061 > > > > > > > > The git tag is release-0.13.1-rc0 and signed with the same pgp key: > > > > https://git-wip-us.apache.org/repos/asf?p=samza.git;a=tag;h= > > > > refs/tags/release-0.14.0-rc3 > > > > > > > > Test binaries have been published to Maven's staging repository, and > > are > > > available > > > > here: > > > > https://repository.apache.org/content/repositories/ > orgapachesamza-1036 > > > > > > > > 61 issues have been resolved as part of this release > > > > https://issues.apache.org/jira/browse/SAMZA-1519?jql= > > > > project%20%3D%20SAMZA%20AND%20fixVersion%20%3D%200.14.0% > > > > 20AND%20status%20%3D%20Resolved > > > > > > > > The vote will be open for 72 hours (ending at 11:00 AM Thursday, > > > > 12/21/2017). > > > > > > > > Please download the release candidate, check the hashes/signature, > > build > > > > it and test it, and then please vote: > > > > > > > > [ ] +1 approve > > > > > > > > [ ] +0 no opinion > > > > > > > > [ ] -1 disapprove (and reason why) > > > > > > > > Thanks, > > > > Xinyu > > > > > > > > > >
