Hi all, I looked at https://issues.apache.org/jira/browse/BEAM-8303, and it seems like the user has a workaround - is that correct? If that's the case, then I vote +1.
@Max - lmk if you'd like to discuss further, but for now my vote is on +1. Best -P. On Fri, Oct 4, 2019 at 9:29 AM Mark Liu <mark...@google.com> wrote: > +1 (forgot to vote) > > I also triggered Java Nexmark on direct, dataflow, spark and flink runner. > Didn't saw performance regression from the dashboard ( > https://apache-beam-testing.appspot.com/dashboard-admin) > > On Fri, Oct 4, 2019 at 8:23 AM Mark Liu <mark...@google.com> wrote: > >> Thanks for the validation work! I validated following: >> >> - Java Quickstart on direct, dataflow,spark local, flink local runner >> - Java mobile gaming on direct and dataflow runner >> - Python Quickstart in batch and streaming in py2/3.5/3.6/3.7 using >> wheals/zip >> - Python Mobile Game in batch/streaming in py2/3.5/3.6/3.7 using >> wheals/zip on direct and dataflow runner >> >> Mark >> >> On Thu, Oct 3, 2019 at 6:57 PM Ahmet Altay <al...@google.com> wrote: >> >>> I see most of the release validations have been completed and marked in >>> the spreadsheet. Thank you all for doing that. If you have not >>> validated/voted yet please take a look at the release candidate. >>> >>> On Thu, Oct 3, 2019 at 7:59 AM Thomas Weise <t...@apache.org> wrote: >>> >>>> I think there is a different reason why the release manager should >>>> probably merge/approve all PRs that go into the release branch while the >>>> release is in progress: >>>> >>>> If/when the need arises for another RC, then only those changes should >>>> be included that are deemed blockers or explicitly agreed. Otherwise the >>>> release can potentially be delayed by modifications that invalidate prior >>>> verification or introduce new instability. >>>> >>> >>> I agree with this reasoning. It expresses my concern in a more clear >>> way. >>> >>> >>>> >>>> Thomas >>>> >>>> >>>> On Thu, Oct 3, 2019 at 3:12 AM Maximilian Michels <m...@apache.org> >>>> wrote: >>>> >>>>> > For the next time, may I suggest asking release manager to do the >>>>> > merging to the release branch. We do not know whether there will be >>>>> an >>>>> > RC2 or not. And if there will not be an RC2 release branch as of now >>>>> > does not directly correspond to what will be released. >>>>> >>>>> The ground truth for releases are the release tags, not the release >>>>> branches. Downstream projects should not depend on the release >>>>> branches. >>>>> Release branches are merely important for the process of creating a >>>>> release, but they lose validity after the RC has been created and >>>>> released. >>>>> >>>>> On 02.10.19 11:45, Ahmet Altay wrote: >>>>> > +1 (validated python quickstarts). Thank you Mark. >>>>> > >>>>> > On Wed, Oct 2, 2019 at 10:49 AM Maximilian Michels <m...@apache.org >>>>> > <mailto:m...@apache.org>> wrote: >>>>> > >>>>> > Thanks for preparing the release, Mark! I would like to address >>>>> > https://issues.apache.org/jira/browse/BEAM-8303 in the release. >>>>> I've >>>>> > already merged the fix to the release-2.16.0 branch. If we do >>>>> another >>>>> > RC, we could include it. As a user is blocked on this, I would >>>>> not vote >>>>> > +1 for this RC, but I also do not want to block the release >>>>> process. >>>>> > >>>>> > >>>>> > Max, thank you for the clear communication for the importance and at >>>>> the >>>>> > same time non-blocking status of the issue. >>>>> > >>>>> > For the next time, may I suggest asking release manager to do the >>>>> > merging to the release branch. We do not know whether there will be >>>>> an >>>>> > RC2 or not. And if there will not be an RC2 release branch as of now >>>>> > does not directly correspond to what will be released. >>>>> > >>>>> > >>>>> > On 01.10.19 09:18, Mark Liu wrote: >>>>> > > Hi everyone, >>>>> > > >>>>> > > Please review and vote on the release candidate #1 for the >>>>> version >>>>> > > 2.16.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 to be deployed to >>>>> > dist.apache.org <http://dist.apache.org> >>>>> > > <http://dist.apache.org> [2], which is signed with the key >>>>> with >>>>> > > fingerprint C110B1C82074883A4241D977599D6305FF3ABB32 [3], >>>>> > > * all artifacts to be deployed to the Maven Central >>>>> Repository [4], >>>>> > > * source code tag ""v2.16.0-RC1" [5], >>>>> > > * website pull request listing the release [6], publishing >>>>> the API >>>>> > > reference manual [7], and the blog post [8]. >>>>> > > * Python artifacts are deployed along with the source release >>>>> to the >>>>> > > dist.apache.org <http://dist.apache.org> < >>>>> http://dist.apache.org> >>>>> > [2]. >>>>> > > * Validation sheet with a tab for 2.16.0 release to help with >>>>> > validation >>>>> > > [9]. >>>>> > > * Docker images published to Docker Hub [10]. >>>>> > > >>>>> > > The vote will be open for at least 72 hours. It is adopted by >>>>> > majority >>>>> > > approval, with at least 3 PMC affirmative votes. >>>>> > > >>>>> > > Thanks, >>>>> > > Mark Liu, Release Manager >>>>> > > >>>>> > > [1] >>>>> > > >>>>> > >>>>> https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12319527&version=12345494 >>>>> > > [2] https://dist.apache.org/repos/dist/dev/beam/2.16.0/ >>>>> > > [3] https://dist.apache.org/repos/dist/release/beam/KEYS >>>>> > > [4] >>>>> > >>>>> https://repository.apache.org/content/repositories/orgapachebeam-1085/ >>>>> > > [5] https://github.com/apache/beam/tree/v2.16.0-RC1 >>>>> > > [6] https://github.com/apache/beam/pull/9667 >>>>> > > [7] https://github.com/apache/beam-site/pull/593 >>>>> > > [8] https://github.com/apache/beam/pull/9671 >>>>> > > [9] >>>>> > > >>>>> > >>>>> https://docs.google.com/spreadsheets/d/1qk-N5vjXvbcEk68GjbkSZTR8AGqyNUM-oLFo_ZXBpJw/edit#gid=890914284 >>>>> > > [10] https://hub.docker.com/u/apachebeam >>>>> > >>>>> >>>>