+1 (non-) Ran Python wordcount on Flink and Spark.
On Wed, Apr 21, 2021 at 5:20 PM Brian Hulette <bhule...@google.com> wrote: > +1 (non-binding) > > I ran a python pipeline exercising the DataFrame API, and another > exercising SQLTransform in Python, both on Dataflow. > > On Wed, Apr 21, 2021 at 12:55 PM Kenneth Knowles <k...@apache.org> wrote: > >> Since the artifacts were changed about 26 hours ago, I intend to leave >> this vote open until 46 hours from now. Specifically, around noon my time >> (US Pacific) on Friday I will close the vote and finalize the release, if >> no problems are discovered. >> >> Kenn >> >> On Wed, Apr 21, 2021 at 12:52 PM Kenneth Knowles <k...@apache.org> wrote: >> >>> +1 (binding) >>> >>> I ran the script at >>> https://beam.apache.org/contribute/release-guide/#run-validations-using-run_rc_validationsh >>> except for the part that requires a GitHub PR, since Cham already did that >>> part. >>> >>> Kenn >>> >>> On Wed, Apr 21, 2021 at 12:11 PM Valentyn Tymofieiev < >>> valen...@google.com> wrote: >>> >>>> +1, verified that my previous findings are fixed. >>>> >>>> On Wed, Apr 21, 2021 at 8:17 AM Chamikara Jayalath < >>>> chamik...@google.com> wrote: >>>> >>>>> +1 (binding) >>>>> >>>>> Ran some Python scenarios and updated the spreadsheet. >>>>> >>>>> Thanks, >>>>> Cham >>>>> >>>>> On Tue, Apr 20, 2021 at 3:39 PM Kenneth Knowles <k...@apache.org> >>>>> wrote: >>>>> >>>>>> >>>>>> >>>>>> On Tue, Apr 20, 2021 at 3:24 PM Robert Bradshaw <rober...@google.com> >>>>>> wrote: >>>>>> >>>>>>> The artifacts and signatures look good to me. +1 (binding) >>>>>>> >>>>>>> (The release branch still has the .dev name, maybe you didn't push? >>>>>>> https://github.com/apache/beam/blob/release-2.29.0/sdks/python/apache_beam/version.py >>>>>>> ) >>>>>>> >>>>>> >>>>>> Good point. I'll highlight that I finally implemented the branching >>>>>> changes from >>>>>> https://lists.apache.org/thread.html/205472bdaf3c2c5876533750d417c19b0d1078131a3dc04916082ce8%40%3Cdev.beam.apache.org%3E >>>>>> >>>>>> The new guide with diagram is here: >>>>>> https://beam.apache.org/contribute/release-guide/#tag-a-chosen-commit-for-the-rc >>>>>> >>>>>> TL;DR: >>>>>> - the release branch continues to be dev/SNAPSHOT for 2.29.0 while >>>>>> the main branch is now dev/SNAPSHOT for 2.30.0 >>>>>> - the RC tag v2.29.0-RC1 no longer lies on the release branch. It is >>>>>> a single tagged commit that removes the dev/SNAPSHOT suffix >>>>>> >>>>>> Kenn >>>>>> >>>>>> >>>>>>> On Tue, Apr 20, 2021 at 10:36 AM Kenneth Knowles <k...@apache.org> >>>>>>> wrote: >>>>>>> >>>>>>>> Please take another look. >>>>>>>> >>>>>>>> - I re-ran the RC creation script so the source release and wheels >>>>>>>> are new and built from the RC tag. I confirmed the source zip and >>>>>>>> wheels >>>>>>>> have version 2.29.0 (not .dev or -SNAPSHOT). >>>>>>>> - I fixed and rebuilt Dataflow worker container images from >>>>>>>> exactly the RC commit, added dataclasses, with internal changes to get >>>>>>>> the >>>>>>>> version to match. >>>>>>>> - I confirmed that the staged jars already have version 2.29.0 >>>>>>>> (not -SNAPSHOT). >>>>>>>> - I confirmed with `diff -r -q` that the source tarball matches >>>>>>>> the RC tag (minus the .git* files and directories and gradlew) >>>>>>>> >>>>>>>> Kenn >>>>>>>> >>>>>>>> On Mon, Apr 19, 2021 at 9:19 PM Kenneth Knowles <k...@apache.org> >>>>>>>> wrote: >>>>>>>> >>>>>>>>> At this point, the release train has just about come around to >>>>>>>>> 2.30.0 which will pick up that change. I don't think it makes sense to >>>>>>>>> cherry-pick anything more into 2.29.0 unless it is nonfunctional. As >>>>>>>>> it is, >>>>>>>>> I think we have a good commit and just need to build the expected >>>>>>>>> artifacts. Since it isn't all the artifacts, I was planning on just >>>>>>>>> overwriting the RC1 artifacts in question and re-verify. I could also >>>>>>>>> roll >>>>>>>>> a new RC2 from the same commit fairly easily. >>>>>>>>> >>>>>>>>> Kenn >>>>>>>>> >>>>>>>>> On Mon, Apr 19, 2021 at 8:57 PM Reuven Lax <re...@google.com> >>>>>>>>> wrote: >>>>>>>>> >>>>>>>>>> Any chance we could include >>>>>>>>>> https://github.com/apache/beam/pull/14548? >>>>>>>>>> >>>>>>>>>> On Mon, Apr 19, 2021 at 8:54 PM Kenneth Knowles <k...@apache.org> >>>>>>>>>> wrote: >>>>>>>>>> >>>>>>>>>>> To clarify: I am running and fixing the release scripts on the >>>>>>>>>>> `master` branch. They work from fresh clones of the RC tag so this >>>>>>>>>>> should >>>>>>>>>>> work in most cases. The exception is the GitHub Actions >>>>>>>>>>> configuration, >>>>>>>>>>> which I cherrypicked >>>>>>>>>>> to the release branch. >>>>>>>>>>> >>>>>>>>>>> Kenn >>>>>>>>>>> >>>>>>>>>>> On Mon, Apr 19, 2021 at 8:34 PM Kenneth Knowles <k...@apache.org> >>>>>>>>>>> wrote: >>>>>>>>>>> >>>>>>>>>>>> OK it sounds like I need to re-roll the artifacts in question. >>>>>>>>>>>> I don't think anything raised here indicates a problem with the >>>>>>>>>>>> tagged >>>>>>>>>>>> commit, but with the state of the release scripts at the time I >>>>>>>>>>>> built the >>>>>>>>>>>> earlier artifacts. >>>>>>>>>>>> >>>>>>>>>>>> On Mon, Apr 19, 2021 at 1:03 PM Robert Bradshaw < >>>>>>>>>>>> rober...@google.com> wrote: >>>>>>>>>>>> >>>>>>>>>>>>> It looks like the wheels are also versioned "2.29.0.dev". >>>>>>>>>>>>> >>>>>>>>>>>>> Not sure if it's important, but the source tarball also seems >>>>>>>>>>>>> to contain some release script changes that are not reflected in >>>>>>>>>>>>> the github >>>>>>>>>>>>> branch. >>>>>>>>>>>>> >>>>>>>>>>>>> On Mon, Apr 19, 2021 at 8:41 AM Kenneth Knowles < >>>>>>>>>>>>> k...@apache.org> wrote: >>>>>>>>>>>>> >>>>>>>>>>>>>> Thanks for the details, Valentyn & Cham. I will fix the >>>>>>>>>>>>>> Dataflow worker containers then update this thread. >>>>>>>>>>>>>> >>>>>>>>>>>>>> Kenn >>>>>>>>>>>>>> >>>>>>>>>>>>>> On Mon, Apr 19, 2021 at 8:36 AM Kenneth Knowles < >>>>>>>>>>>>>> k...@apache.org> wrote: >>>>>>>>>>>>>> >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> On Fri, Apr 16, 2021 at 3:42 AM Elliotte Rusty Harold < >>>>>>>>>>>>>>> elh...@ibiblio.org> wrote: >>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> On Fri, Apr 16, 2021 at 4:02 AM Kenneth Knowles < >>>>>>>>>>>>>>>> k...@apache.org> wrote: >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> > The complete staging area is available for your review, >>>>>>>>>>>>>>>> which includes: >>>>>>>>>>>>>>>> > * JIRA release notes [1], >>>>>>>>>>>>>>>> > * the official Apache source release to be deployed to >>>>>>>>>>>>>>>> dist.apache.org [2], which is signed with the key with >>>>>>>>>>>>>>>> fingerprint 03DBA3E6ABDD04BFD1558DC16ED551A8AE02461C [3], >>>>>>>>>>>>>>>> > * all artifacts to be deployed to the Maven Central >>>>>>>>>>>>>>>> Repository [4], >>>>>>>>>>>>>>>> > * source code tag "v2.29.0-RC1" [5], >>>>>>>>>>>>>>>> > * website pull request listing the release [6], >>>>>>>>>>>>>>>> publishing the API reference manual [7], and the blog post [8]. >>>>>>>>>>>>>>>> > * Java artifacts were built with Maven MAVEN_VERSION and >>>>>>>>>>>>>>>> OpenJDK/Oracle JDK JDK_VERSION. >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> Are the MAVEN_VERSION and OpenJDK/Oracle JDK JDK_VERSION >>>>>>>>>>>>>>>> supposed to >>>>>>>>>>>>>>>> be filled in with numbers? >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> Yes, I missed that these were variables to be replaced. >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> JDK_VERSION=8u181 (1.8) and the Gradle version is taken from >>>>>>>>>>>>>>> the gradlew config so no need to include in the template, but >>>>>>>>>>>>>>> it is 6.8 >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> Kenn >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> -- >>>>>>>>>>>>>>>> Elliotte Rusty Harold >>>>>>>>>>>>>>>> elh...@ibiblio.org >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>