+1 (based on looking at the diff) Glad to see that ended up pinning to a range of known working versions, as opposed to simply bumping the new lower bound.
On Thu, Sep 21, 2017 at 10:21 AM, Kenneth Knowles <k...@google.com.invalid> wrote: > +1 > > On Thu, Sep 21, 2017 at 10:15 AM, Chamikara Jayalath <chamik...@apache.org > > > wrote: > > > +1. > > > > Ran wordcount and verified checksums and signature. > > > > Thanks, > > Cham > > > > On Thu, Sep 21, 2017 at 1:02 AM Robert Bradshaw > > <rober...@google.com.invalid> > > wrote: > > > > > Hi everyone, > > > > > > As discussed earlier in this list [1] we'd like to get a bugfix > > > release out for beam 2.1. Please review and vote on the release > > > candidate #1 for the version 2.1.1, as follows: > > > > > > [ ] +1, Approve the release > > > [ ] -1, Do not approve the release (please provide specific comments) > > > > > > Artifacts are at [2] and the full diff can be viewed at [3] (two > > > cherry picks and a version bump). > > > > > > The vote will be open for at least 36 hours. It is adopted by majority > > > approval, with at least 3 PMC affirmative votes. > > > > > > Thanks, > > > Robert > > > > > > > > > [1] > > > http://mail-archives.apache.org/mod_mbox/incubator-beam- > > dev/201709.mbox/browser > > > [2] https://dist.apache.org/repos/dist/dev/beam/2.1.1/ > > > [3] https://github.com/apache/beam/compare/release-2.1.0... > release-2.1.1 > > > > > >
