-1, the wheel files seem to be built against the wrong commit. E.g.

unzip -p
apache_beam-2.21.0-cp35-cp35m-macosx_10_6_intel.macosx_10_9_intel.macosx_10_9_x86_64.macosx_10_10_intel.macosx_10_10_x86_64.whl
apache_beam/runners/worker/bundle_processor.py | head -n 40

notice the missing "import bisect" (among other things) missing from
https://github.com/apache/beam/blob/release-2.21.0/sdks/python/apache_beam/runners/worker/bundle_processor.py
.

(I do agree that BEAM-9887 isn't severe enough to hold up the release at
this point.)


On Tue, May 19, 2020 at 8:48 PM rahul patwari <rahulpatwari8...@gmail.com>
wrote:

> Hi Luke,
>
> The release is not severely broken without PR #11609.
> The PR ensures that, while building a Row with Logical Type, the input
> value provided is proper. If we take FixedBytes logical type with length
> 10, for example, the proper input value will be a byte array of length 10.
> But, without this PR, for FixedBytes logical type, the Row will be built
> with input value with length less than the expected length.
> But, as long as the input value provided is correct, there shouldn't be
> any problems.
> I will change the fix version as 2.22.0 for BEAM-9887
> <https://issues.apache.org/jira/browse/BEAM-9887>.
>
> Regards,
> Rahul
>
> On Wed, May 20, 2020 at 8:51 AM Luke Cwik <lc...@google.com> wrote:
>
>> Rahul, do you believe that the release is severely broken without
>> PR/11609 enough to require another release candidate or would waiting till
>> 2.22 (which is due to be cut tomorrow)?
>>
>> On Tue, May 19, 2020 at 8:13 PM rahul patwari <rahulpatwari8...@gmail.com>
>> wrote:
>>
>>> Hi,
>>>
>>> Can the PR: https://github.com/apache/beam/pull/11609 be cherry-picked
>>> for 2.21.0 release?
>>> If not, the fix version has to be changed for BEAM-9887
>>> <https://issues.apache.org/jira/browse/BEAM-9887>.
>>>
>>> Regards,
>>> Rahul
>>>
>>> On Wed, May 20, 2020 at 6:05 AM Ahmet Altay <al...@google.com> wrote:
>>>
>>>> +1, I validated python 2 and 3 quickstarts.
>>>>
>>>> On Tue, May 19, 2020 at 4:57 PM Hannah Jiang <hannahji...@google.com>
>>>> wrote:
>>>>
>>>>> I confirmed that licenses/notices/source code are added to Java and
>>>>> Python docker images as expected.
>>>>>
>>>>>
>>>>> On Tue, May 19, 2020 at 2:36 PM Kyle Weaver <kcwea...@google.com>
>>>>> wrote:
>>>>>
>>>>>> Thanks for bringing that up Steve. I'll leave it to others to vote on
>>>>>> whether that necessitates an RC #2.
>>>>>>
>>>>>> On Tue, May 19, 2020 at 5:22 PM Steve Niemitz <sniem...@apache.org>
>>>>>> wrote:
>>>>>>
>>>>>>> https://issues.apache.org/jira/browse/BEAM-10015 was marked as 2.21
>>>>>>> but isn't in the RC1 tag.  It's marked as P1, and seems like the
>>>>>>> implication is that without the fix, pipelines can produce incorrect 
>>>>>>> data.
>>>>>>> Is this a blocker?
>>>>>>>
>>>>>>
>>>> +Reuven Lax <re...@google.com>, would this be a release blocker?
>>>>
>>>>
>>>>>
>>>>>>> On Tue, May 19, 2020 at 4:51 PM Kyle Weaver <kcwea...@google.com>
>>>>>>> wrote:
>>>>>>>
>>>>>>>> Hi everyone,
>>>>>>>> Please review and vote on the release candidate #1 for the version
>>>>>>>> 2.21.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 [2], which is signed with the key with fingerprint
>>>>>>>> F11E37D7F006D086232876797B6D6673C79AEA72 [3],
>>>>>>>> * all artifacts to be deployed to the Maven Central Repository [4],
>>>>>>>> * source code tag "v2.21.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 3.6.3 and OpenJDK/Oracle JDK
>>>>>>>> 1.8.0.
>>>>>>>> * Python artifacts are deployed along with the source release to
>>>>>>>> the dist.apache.org [2].
>>>>>>>> * Validation sheet with a tab for 2.21.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,
>>>>>>>> Kyle
>>>>>>>>
>>>>>>>> [1]
>>>>>>>> https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12319527&version=12347143
>>>>>>>> [2] https://dist.apache.org/repos/dist/dev/beam/2.21.0/
>>>>>>>> [3] https://dist.apache.org/repos/dist/release/beam/KEYS
>>>>>>>> [4]
>>>>>>>> https://repository.apache.org/content/repositories/orgapachebeam-1103/
>>>>>>>> [5] https://github.com/apache/beam/releases/tag/v2.21.0-RC1
>>>>>>>> [6] https://github.com/apache/beam/pull/11727
>>>>>>>> [7] https://github.com/apache/beam-site/pull/603
>>>>>>>> [8] https://github.com/apache/beam/pull/11729
>>>>>>>> [9]
>>>>>>>> https://docs.google.com/spreadsheets/d/1qk-N5vjXvbcEk68GjbkSZTR8AGqyNUM-oLFo_ZXBpJw/edit#gid=275707202
>>>>>>>> [10] https://hub.docker.com/search?q=apache%2Fbeam&type=image
>>>>>>>>
>>>>>>>

Reply via email to