[ 
https://issues.apache.org/jira/browse/BEAM-27?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15590118#comment-15590118
 ] 

ASF GitHub Bot commented on BEAM-27:
------------------------------------

GitHub user kennknowles opened a pull request:

    https://github.com/apache/incubator-beam/pull/1136

    [BEAM-27] Add analysis of Timer parameters for DoFn.ProcessElement

    Be sure to do all of the following to help us incorporate your contribution
    quickly and easily:
    
     - [x] Make sure the PR title is formatted like:
       `[BEAM-<Jira issue #>] Description of pull request`
     - [x] Make sure tests pass via `mvn clean verify`. (Even better, enable
           Travis-CI on your fork and ensure the whole test matrix passes).
     - [x] Replace `<Jira issue #>` in the title with the actual Jira issue
           number, if there is one.
     - [x] If this contribution is large, please file an Apache
           [Individual Contributor License 
Agreement](https://www.apache.org/licenses/icla.txt).
    
    ---
    
    This PR adds `Timer` parameters. It includes #1102 and #1130. The final 
commit is the only one that is new, and can be reviewed independently.
    
    R: @jkff 
    
    I will pull in a committer later.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/kennknowles/incubator-beam Timer-params

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/incubator-beam/pull/1136.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1136
    
----
commit 507cba8f0df44b1c11c7621f00ff492243bdd1f1
Author: Kenneth Knowles <k...@google.com>
Date:   2016-10-14T06:18:40Z

    Tighten up tests for StateId validation

commit 08fdb304a777d981fa3860fcd0270aa40f99da75
Author: Kenneth Knowles <k...@google.com>
Date:   2016-10-14T21:19:59Z

    Fix condition on state/splittable ParDo validation

commit 65eaedbfb628f6eb42d17c16220e973b064608d7
Author: Kenneth Knowles <k...@google.com>
Date:   2016-10-13T20:41:38Z

    Add Timer, TimerId, TimerSpec and TimerSpecs
    
    These are the simple specifications that describe
    a timer without instantiating it. Consists only of
    the time domain.

commit 0898999f34270a11beddb68e5068c5258ec9ebec
Author: Kenneth Knowles <k...@google.com>
Date:   2016-10-14T02:53:31Z

    Add DoFnSignatures analysis of timer declarations

commit e44e56da62e48f1d763e2b2d46d55a04fbe3596c
Author: Kenneth Knowles <k...@google.com>
Date:   2016-10-18T20:09:57Z

    Reject all timers in ParDo, for now

commit bcb07b534ffff475637d06a14695f38b1e151912
Author: Kenneth Knowles <k...@google.com>
Date:   2016-10-18T21:15:34Z

    fixup! More caveats on Timer javadoc

commit 97aaf9730915d426fd865eeb67793988a8dbdefb
Author: Kenneth Knowles <k...@google.com>
Date:   2016-10-18T21:16:09Z

    fixup! state javadoc

commit ac419d245a7bd3dc9263e61567676e3d92133f9a
Author: Kenneth Knowles <k...@google.com>
Date:   2016-10-18T21:21:41Z

    fixup! refine Timer javadoc

commit 28fbe1890b54e26f7c0d0bc547440af061d1e907
Author: Kenneth Knowles <k...@google.com>
Date:   2016-10-18T22:52:28Z

    fixup! validate is now void

commit dcce856a1fa3d8bde6b47cf200b4444d212de9d2
Author: Kenneth Knowles <k...@google.com>
Date:   2016-10-14T17:51:57Z

    Use the correct State class in DoFnSignatures

commit 70ec03e5cacd272b194589ab3d7ce89e39be7b13
Author: Kenneth Knowles <k...@google.com>
Date:   2016-10-14T18:07:55Z

    Add analysis and validation of State parameters to DoFn.ProcessElement

commit 5883e0c087f1b5642f9349ab099f2eaa82caf8fa
Author: Kenneth Knowles <k...@google.com>
Date:   2016-10-19T22:00:30Z

    Merge remote-tracking branch 'github/pr/1102' into Timer-params

commit bda251e05ffe48873ccaf06f005c3847acd6085b
Author: Kenneth Knowles <k...@google.com>
Date:   2016-10-19T22:02:36Z

    Add analysis of Timer parameters to DoFn.ProcessElement

----


> Add user-ready API for interacting with timers
> ----------------------------------------------
>
>                 Key: BEAM-27
>                 URL: https://issues.apache.org/jira/browse/BEAM-27
>             Project: Beam
>          Issue Type: Sub-task
>          Components: sdk-java-core
>            Reporter: Kenneth Knowles
>            Assignee: Kenneth Knowles
>
> Pipeline authors will benefit from a different factorization of interaction 
> with underlying timers. The current APIs are targeted at runner implementers.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to