+1 (binding) Carried over from previous vote on earlier RC, with the following additional tests:
- Verified a few upgrade scenarios for the key / namespace serializer in RocksDB state backend - Verified checksums and signatures - Built source (without Hadoop, Scala 2.11 + Scala 2.12) with success - Ran end-to-end tests locally Cheers, Gordon On Mon, Apr 8, 2019 at 5:36 AM Thomas Weise <t...@apache.org> wrote: > +1 (binding) > > * verified checksums and signatures > * run release build, built internal packages, run few internal tests > > Couple observations: > > Table API artifact change: > https://issues.apache.org/jira/browse/FLINK-11064 > > JIRA mentions the change, but should we be more explicit and add that info > into the release blog/notes? > > The version number for shaded hadoop2 surprised me: > > Example: flink-shaded-hadoop2-2.8.3-1.8.0.jar > > If 2.8.3 is part of the version number (vs. artifactId), then shouldn't the > version number be 1.8.0-2.8.3? (Alternatively, the hadoop version could be > included into artifactId.) > > Thomas > > On Sun, Apr 7, 2019 at 12:15 PM Chesnay Schepler <ches...@apache.org> > wrote: > > > Missed that one :/ > > > > On 06/04/2019 08:29, Aljoscha Krettek wrote: > > > Ah, the first two are not on 1.8.0-rc5, but FLINK-11855 is. > > > > > >> On 6. Apr 2019, at 08:23, Aljoscha Krettek <aljos...@apache.org> > wrote: > > >> > > >> Thanks, Chesnay! I had that tab open already and was prepared to do it > > but it’s good you also thought about it. :-) What about these three > issues, > > though: > > > https://issues.apache.org/jira/issues/?filter=12334772&jql=project%20%3D%20FLINK%20AND%20fixVersion%20%3D%201.8.1%20and%20resolution%20!%3D%20unresolved%20ORDER%20BY%20updated%20DESC > > < > > > https://issues.apache.org/jira/issues/?filter=12334772&jql=project%20=%20FLINK%20AND%20fixVersion%20=%201.8.1%20and%20resolution%20!=%20unresolved%20ORDER%20BY%20updated%20DESC > > > > > >> > > >> Aljoscha > > >> > > >>> On 5. Apr 2019, at 14:05, Chesnay Schepler <ches...@apache.org> > wrote: > > >>> > > >>> Went through JIRA and update the versions. > > >>> > > >>> On 05/04/2019 13:57, Chesnay Schepler wrote: > > >>>> +1 > > >>>> > > >>>> We'll have to comb through JIRA again as I already found a number of > > tickets that were marked as fixed for 1.8.1, but are included in this RC. > > >>>> > > >>>> On 05/04/2019 07:10, Rong Rong wrote: > > >>>>> +1 (non-binding) > > >>>>> > > >>>>> * Verified checksums and GPG files matches release files > > >>>>> * Verified that the source archives do not contain any binaries > > >>>>> * Built the source with Maven to ensure all source files have > Apache > > headers > > >>>>> * Checked that all POM files point to the same version > > >>>>> * `mvn clean verify` against scala-2.11 and scala-2.12 > > >>>>> - Also verified rest.port setting is taking effect after > > >>>>> YarnClusterEntryUtils change > > >>>>> * Verified quickstart-scala and quickstart-java are working with > the > > staging > > >>>>> repository. > > >>>>> > > >>>>> -- > > >>>>> Rong > > >>>>> > > >>>>> > > >>>>> On Wed, Apr 3, 2019 at 11:22 PM Aljoscha Krettek < > > aljos...@apache.org> > > >>>>> wrote: > > >>>>> > > >>>>>> Hi everyone, > > >>>>>> Please review and vote on the release candidate 5 for Flink 1.8.0, > > as > > >>>>>> follows: > > >>>>>> [ ] +1, Approve the release > > >>>>>> [ ] -1, Do not approve the release (please provide specific > > comments) > > >>>>>> > > >>>>>> > > >>>>>> The complete staging area is available for your review, which > > includes: > > >>>>>> * JIRA release notes [1], > > >>>>>> * the official Apache source release and binary convenience > > releases to be > > >>>>>> deployed to dist.apache.org [2], which are signed with the key > with > > >>>>>> fingerprint F2A67A8047499BBB3908D17AA8F4FD97121D7293 [3], > > >>>>>> * all artifacts to be deployed to the Maven Central Repository > [4], > > >>>>>> * source code tag "release-1.8.0-rc5" [5], > > >>>>>> * website pull request listing the new release [6] > > >>>>>> * website pull request adding announcement blog post [7]. > > >>>>>> > > >>>>>> The vote will be open for at least 72 hours. It is adopted by > > majority > > >>>>>> approval, with at least 3 PMC affirmative votes. > > >>>>>> > > >>>>>> Thanks, > > >>>>>> Aljoscha > > >>>>>> > > >>>>>> [1] > > >>>>>> > > > https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12315522&version=12344274 > > >>>>>> [2] https://dist.apache.org/repos/dist/dev/flink/flink-1.8.0-rc5/ > > >>>>>> [3] https://dist.apache.org/repos/dist/release/flink/KEYS > > >>>>>> [4] > > https://repository.apache.org/content/repositories/orgapacheflink-1216 > > >>>>>> [5] > > >>>>>> > > > https://gitbox.apache.org/repos/asf?p=flink.git;a=tag;h=33943b15f9eca3fafbd88b9315cf9f0a083b6381 > > >>>>>> [6] https://github.com/apache/flink-web/pull/180 > > >>>>>> [7] https://github.com/apache/flink-web/pull/179 > > >>>>>> > > >>>>>> P.S. The difference to the previous RCs is small, you can fetch > the > > tags > > >>>>>> and do a "git log release-1.8.0-rc1..release-1.8.0-rc5” to see the > > >>>>>> difference in commits. Its fixes for the issues that led to the > > >>>>>> cancellation of the previous RCs plus smaller fixes. Most > > >>>>>> verification/testing that was carried out should apply as is to > > this RC. > > >>>>>> Any functional verification that you did on previous RCs should > > therefore > > >>>>>> easily carry over to this one. This does add one new feature for > > the Kafka > > >>>>>> connector that was on master for quite a while but not on > > release-1.8 > > >>>>>> because of discussions. The NOTICE-binary file is updated to > > reflect the > > >>>>>> slight change in ordering; shaded-guava now appears at a different > > part in > > >>>>>> the dependency tree. > > >>>> > > >>>> > > > > > > > >