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 <mailto: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 
> <mailto:azert...@gmail.com>> wrote:
> Adapt documentation:
> 
> https://beam.apache.org/documentation/transforms/python/elementwise/pardo/#example-1-pardo-with-a-simple-dofn
>  
> <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