@Pablo Thanks for asking. Since the user is satisfied with the workaround for now, I feel better about the release. It's also good to stay on track with the release schedule.

@Ahmet/@Thomas Makes sense to go through the release manager, as it prevents confusion around what is currently in the release branch.

On 04.10.19 10:17, Pablo Estrada wrote:
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 <mailto: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
    <mailto: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
        <mailto: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
            <mailto: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 <mailto: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>
                     > <mailto: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>
                     >      > <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> <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
                     >

Reply via email to