Re: Upcoming potentially breaking change to CoGroupByKey
"new release" is an ambiguous descriptor in this email. I'm going to
continue to take it as "right after the 2.42.0 cut, to make it into the
2.43.0 as planned".
On Tue, Sep 6, 2022 at 2:39 PM Ryan Thompson
wrote:
> There was discussion in the python meeting to try to get this into the new
> release. The consensus was that putting it in right after the release had
> the highest chance of catching problems with the least amount of pain.
>
> I can send it out to users next week if nother halts the change.
>
> On Tue, Sep 6, 2022 at 5:03 PM Luke Cwik wrote:
>
>> We should send this out to us...@beam.apache.org so that they are aware
>> of this change once commenting in the doc has settled.
>>
>> On Tue, Sep 6, 2022 at 1:59 PM Robert Burke wrote:
>>
>>> Thank you for already planning to *NOT* have this merged until after
>>> this week's 2.42.0 cut. This Release Manager is pleased that the doc says
>>> it's intended for 2.43.0.
>>>
>>>
>>> On Tue, Sep 6, 2022, 1:44 PM Ryan Thompson via dev
>>> wrote:
>>>
CoGroupByKey returns a dictionary of {KeyType, List[ValueType]} but, as
GroupByKey, should return an Iterable.
Change:
https://github.com/apache/beam/pull/22984
Please look at this doc
if you need more details. Feel free to comment.
Re: Upcoming potentially breaking change to CoGroupByKey
There was discussion in the python meeting to try to get this into the new
release. The consensus was that putting it in right after the release had
the highest chance of catching problems with the least amount of pain.
I can send it out to users next week if nother halts the change.
On Tue, Sep 6, 2022 at 5:03 PM Luke Cwik wrote:
> We should send this out to us...@beam.apache.org so that they are aware
> of this change once commenting in the doc has settled.
>
> On Tue, Sep 6, 2022 at 1:59 PM Robert Burke wrote:
>
>> Thank you for already planning to *NOT* have this merged until after this
>> week's 2.42.0 cut. This Release Manager is pleased that the doc says it's
>> intended for 2.43.0.
>>
>>
>> On Tue, Sep 6, 2022, 1:44 PM Ryan Thompson via dev
>> wrote:
>>
>>> CoGroupByKey returns a dictionary of {KeyType, List[ValueType]} but, as
>>> GroupByKey, should return an Iterable.
>>>
>>> Change:
>>> https://github.com/apache/beam/pull/22984
>>>
>>> Please look at this doc
>>>
>>> if you need more details. Feel free to comment.
>>>
>>>
Re: Upcoming potentially breaking change to CoGroupByKey
We should send this out to us...@beam.apache.org so that they are aware of
this change once commenting in the doc has settled.
On Tue, Sep 6, 2022 at 1:59 PM Robert Burke wrote:
> Thank you for already planning to *NOT* have this merged until after this
> week's 2.42.0 cut. This Release Manager is pleased that the doc says it's
> intended for 2.43.0.
>
>
> On Tue, Sep 6, 2022, 1:44 PM Ryan Thompson via dev
> wrote:
>
>> CoGroupByKey returns a dictionary of {KeyType, List[ValueType]} but, as
>> GroupByKey, should return an Iterable.
>>
>> Change:
>> https://github.com/apache/beam/pull/22984
>>
>> Please look at this doc
>>
>> if you need more details. Feel free to comment.
>>
>>
Re: Upcoming potentially breaking change to CoGroupByKey
Thank you for already planning to *NOT* have this merged until after this
week's 2.42.0 cut. This Release Manager is pleased that the doc says it's
intended for 2.43.0.
On Tue, Sep 6, 2022, 1:44 PM Ryan Thompson via dev
wrote:
> CoGroupByKey returns a dictionary of {KeyType, List[ValueType]} but, as
> GroupByKey, should return an Iterable.
>
> Change:
> https://github.com/apache/beam/pull/22984
>
> Please look at this doc
>
> if you need more details. Feel free to comment.
>
>
Upcoming potentially breaking change to CoGroupByKey
CoGroupByKey returns a dictionary of {KeyType, List[ValueType]} but, as
GroupByKey, should return an Iterable.
Change:
https://github.com/apache/beam/pull/22984
Please look at this doc
if you need more details. Feel free to comment.
