I tried building the release candidate(RC1) and it fails with the following checkstyle errors.
[ant:checkstyle] /Users/svenkata/Documents/apache-samza-1.0.0-src/samza-test/src/main/java/org/apache/samza/test/integration/TestStandaloneIntegrationApplication.java:43: 'member def type' have incorrect indentation level 5, expected level should be 4. [ant:checkstyle] /Users/svenkata/Documents/apache-samza-1.0.0-src/samza-test/src/main/java/org/apache/samza/test/integration/TestStandaloneIntegrationApplication.java:43: 'method def' child have incorrect indentation level 5, expected level should be 4. [ant:checkstyle] /Users/svenkata/Documents/apache-samza-1.0.0-src/samza-test/src/main/java/org/apache/samza/test/integration/TestStandaloneIntegrationApplication.java:44: 'method def' child have incorrect indentation level 5, expected level should be 4. Thanks. On Mon, Oct 22, 2018 at 9:09 PM Prateek Maheshwari <prate...@utexas.edu> wrote: > Hi all, > > This is a call for a vote on a release of Apache Samza 1.0.0. Thanks to > everyone who has contributed to this release. > > The release candidate can be downloaded from here: > http://home.apache.org/~pmaheshwari/samza-1.0.0-rc1/ > > The release candidate is signed with pgp key 6585B3D7, which can be found > on keyservers: https://pgp.mit.edu/pks/lookup?op=get&search=0x6585B3D7 > > The git tag is release-1.0.0-rc1 and signed with the same pgp key: > > https://git-wip-us.apache.org/repos/asf?p=samza.git;a=tag;h=refs/tags/release-1.0.0-rc1 > > Test binaries have been published to Maven's staging repository, and are > available here: > https://repository.apache.org/content/repositories/orgapachesamza-1052/ > > The vote will be open for 72 hours (ending at 9:00 PM PST Thursday, > 10/25/2018). > > 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) > > For me, I ran check-all.sh, integration tests and verified the SQL console > in samza-tool tgz. So +1 (non-binding) from my side. > > Thanks, > Prateek >
