[ 
https://issues.apache.org/jira/browse/BEAM-9650?focusedWorklogId=422849&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-422849
 ]

ASF GitHub Bot logged work on BEAM-9650:
----------------------------------------

                Author: ASF GitHub Bot
            Created on: 15/Apr/20 17:08
            Start Date: 15/Apr/20 17:08
    Worklog Time Spent: 10m 
      Work Description: soyrice commented on pull request #11415: [BEAM-9650] 
Cleanup documentation on side inputs patterns
URL: https://github.com/apache/beam/pull/11415#discussion_r408999158
 
 

 ##########
 File path: website/src/documentation/patterns/side-inputs.md
 ##########
 @@ -46,29 +54,34 @@ For instance, the following code sample uses a `Map` to 
create a `DoFn`. The `Ma
 {% github_sample 
/apache/beam/blob/master/examples/java/src/main/java/org/apache/beam/examples/snippets/Snippets.java
 tag:SideInputPatternSlowUpdateGlobalWindowSnip1
 %}
 ```
+```py
+No sample present.
+```
 
 
 ## Slowly updating side input using windowing
 
-You can read side input pcollection periodically into distinct windows.
-Later, when you apply side input to your main input, windows will be matched 
automatically 1:1.
-This way, you can guarantee side input consistency on the duration of the 
single window.
-
-To do this, you can utilize PeriodicSequence PTransform that will generate 
infinite sequence
-of elements with some real-time period:
-
-1. Use the PeriodicImpulse transform to generate windowed periodic sequence.
-
-    a. MAX_TIMESTAMP can be replaced with some closer boundary if you want to 
stop generating elements at some point.
-
-1. Read data using Read operation triggered by arrival of PCollection element.
-
+You can read side input data periodically into distinct PCollection windows.
+Later, when you apply the side input to your main input, each main input
+window is automatically matched to a single side input window.
+This guarantees consistency on the duration of the single window,
+meaning that each window on the main input will be matched to a single
+version of side input data.
+
+Implementation of described approach can be narrowed down to:
+
+1. Use the PeriodicImpulse or PeriodicSequence PTransform to: 
+    * Generate an infinite sequence of elements at required processing time
+    intervals
+    * Assign them to separate windows.
+1. Fetch data using SDF Read or ReadAll PTransform triggered by arrival of
+PCollection element.
 1. Apply side input.
 
 Review comment:
   Missing "the" -> "Apply the side input"
 
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Issue Time Tracking
-------------------

    Worklog Id:     (was: 422849)
    Time Spent: 6h 20m  (was: 6h 10m)

> Add consistent slowly changing side inputs support
> --------------------------------------------------
>
>                 Key: BEAM-9650
>                 URL: https://issues.apache.org/jira/browse/BEAM-9650
>             Project: Beam
>          Issue Type: Bug
>          Components: io-ideas
>            Reporter: Mikhail Gryzykhin
>            Assignee: Mikhail Gryzykhin
>            Priority: Major
>          Time Spent: 6h 20m
>  Remaining Estimate: 0h
>
> Add implementation for slowly changing dimentions based on [design 
> doc](https://docs.google.com/document/d/1LDY_CtsOJ8Y_zNv1QtkP6AGFrtzkj1q5EW_gSChOIvg/edit]



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to