Hey guys

Sorry I haven't had the time to get it done. It's still on my todo list
though! I will find the time this weekend to make the necessary changes.

Sincerely


Kirill Ism

On Fri, Mar 4, 2022, 19:29 Ahmet Altay <al...@google.com> wrote:

> Thank you!
>
> On Fri, Mar 4, 2022 at 10:18 AM David Cavazos <dcava...@google.com> wrote:
>
>> Thanks! Sorry this got lost in my inbox. I opened a PR to update both
>> points.
>>
>> https://github.com/apache/beam/pull/17018
>>
>> On Fri, Mar 4, 2022 at 9:04 AM Ahmet Altay <al...@google.com> wrote:
>>
>>> Hi Kirill, have you had a chance to create a PR? It is ok if you did not
>>> have time.
>>>
>>> @David Cavazos <dcava...@google.com> - Could you please fix the issue
>>> Kirill reported if it is still open?
>>>
>>> On Wed, Feb 9, 2022 at 8:54 AM Ahmet Altay <al...@google.com> wrote:
>>>
>>>> Awesome :) And feel free to add me as the reviewer!
>>>>
>>>> On Wed, Feb 9, 2022 at 4:55 AM Kirill Love <azert...@gmail.com> wrote:
>>>>
>>>>> Dear Ahmet, David
>>>>>
>>>>> That’s a good suggestion! I’ll try to free up some time in the weekend
>>>>> to create a PR for this.
>>>>>
>>>>> Sincerely
>>>>>
>>>>>
>>>>> Kirill Ism
>>>>>
>>>>> On 9 Feb 2022, at 03:15, Ahmet Altay <al...@google.com> wrote:
>>>>>
>>>>> Thank you for the feedback Kirill. If you have time we will
>>>>> happily accept your contribution in the form of a PR :)
>>>>>
>>>>> @David Cavazos <dcava...@google.com> - You are the author of this
>>>>> page. Would you like to make the changes?
>>>>>
>>>>> On Tue, Feb 8, 2022 at 9:20 AM Kirill Love <azert...@gmail.com> wrote:
>>>>>
>>>>>> Adapt documentation:
>>>>>>
>>>>>>
>>>>>> https://beam.apache.org/documentation/transforms/python/elementwise/pardo/#example-1-pardo-with-a-simple-dofn
>>>>>>
>>>>>> ## Can be removed
>>>>>>
>>>>>> [BEAM-7885] <https://issues.apache.org/jira/browse/BEAM-7885>
>>>>>> DoFn.setup() doesn’t run for streaming jobs running in the
>>>>>> DirectRunner.
>>>>>>
>>>>>>
>>>>>> Issue has been resolved.
>>>>>>
>>>>>>
>>>>>> ## Can be changed
>>>>>>
>>>>>> self.window = beam.window.GlobalWindow()
>>>>>>
>>>>>>
>>>>>> New path is:
>>>>>>
>>>>>> beam.transforms.window.GlobalWindow()
>>>>>>
>>>>>>
>>>>>> Sincerely
>>>>>>
>>>>>>
>>>>>> Kirill Ism
>>>>>>
>>>>>>
>>>>>>
>>>>>

Reply via email to