OK, this appears to have been a weird config issue on my system
(though the error certainly could have been better). As BEAM-8303 has
a workaround and all else is looking good, I don't think that's worth
another RC.

+1 (binding) to this release.

On Fri, Oct 4, 2019 at 10:56 AM Robert Bradshaw <rober...@google.com> wrote:
>
> The artifact signatures and contents all look good to me. I've also
> verify the wheels work for the direct runner. However, I'm having an
> issue with trying to run on dataflow with Python 3.6:
>
> python -m apache_beam.examples.wordcount   --input
> gs://clouddfe-robertwb/chicago_taxi_data/eval/data.csv   --output
> gs://clouddfe-robertwb/test/xcounts.txt   --runner=Dataflow
> --project=google.com:clouddfe
> --temp_location=gs://clouddfe-robertwb/fn-api/tmp
> --staging_location=gs://clouddfe-robertwb/tmp
> --sdk_location=staging/apache-beam-2.16.0.zip
> ...
>   File 
> "/usr/local/google/home/robertwb/beam-release/release-verify/staging/test-venv/lib/python3.6/site-packages/apache_beam/io/gcp/gcsio.py",
> line 374, in exists
>     self.client.objects.Get(request)  # metadata
>   File 
> "/usr/local/google/home/robertwb/beam-release/release-verify/staging/test-venv/lib/python3.6/site-packages/apache_beam/io/gcp/internal/clients/storage/storage_v1_client.py",
> line 1100, in Get
>     download=download)
>   File 
> "/usr/local/google/home/robertwb/beam-release/release-verify/staging/test-venv/lib/python3.6/site-packages/apitools/base/py/base_api.py",
> line 729, in _RunMethod
>     http, http_request, **opts)
>   File 
> "/usr/local/google/home/robertwb/beam-release/release-verify/staging/test-venv/lib/python3.6/site-packages/apitools/base/py/http_wrapper.py",
> line 360, in MakeRequest
>     max_retry_wait, total_wait_sec))
>   File 
> "/usr/local/google/home/robertwb/beam-release/release-verify/staging/test-venv/lib/python3.6/site-packages/apache_beam/io/gcp/gcsio_overrides.py",
> line 43, in retry_func
>     return http_wrapper.HandleExceptionsAndRebuildHttpConnections(retry_args)
>   File 
> "/usr/local/google/home/robertwb/beam-release/release-verify/staging/test-venv/lib/python3.6/site-packages/apitools/base/py/http_wrapper.py",
> line 294, in HandleExceptionsAndRebuildHttpConnections
>     retry_args.exc.status >= 500)):
>
> Is this just me or a wider issue?
>
> On Fri, Oct 4, 2019 at 10:27 AM Pablo Estrada <pabl...@google.com> 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> 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
> >>>>>> >

Reply via email to