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

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

                Author: ASF GitHub Bot
            Created on: 14/Apr/20 14:55
            Start Date: 14/Apr/20 14:55
    Worklog Time Spent: 10m 
      Work Description: soyrice commented on pull request #11182: [BEAM-9650] 
Add PeriodicImpulse Transform and slowly changing side input documentation
URL: https://github.com/apache/beam/pull/11182#discussion_r408195934
 
 

 ##########
 File path: website/src/documentation/patterns/side-inputs.md
 ##########
 @@ -45,4 +45,30 @@ For instance, the following code sample uses a `Map` to 
create a `DoFn`. The `Ma
 ```java
 {% github_sample 
/apache/beam/blob/master/examples/java/src/main/java/org/apache/beam/examples/snippets/Snippets.java
 tag:SideInputPatternSlowUpdateGlobalWindowSnip1
 %}
-```
\ No newline at end of file
+```
+
+
+## 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.
 
 Review comment:
   Missing "the" -> "when you apply the side input to your main input"
   
   Present tense -> "windows are automatically matched"
   
   It might be less ambiguously to say "each side input window is matched to a 
main input window" (or something like that), instead of "windows will be 
matched automatically 1:1." I'm not sure it's clear what it means to be 
"matched automatically 1:1."
 
----------------------------------------------------------------
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:
[email protected]


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

    Worklog Id:     (was: 422078)
    Time Spent: 3h 10m  (was: 3h)

> 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: 3h 10m
>  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