You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@beam.apache.org by nedgar <gi...@git.apache.org> on 2017/11/13 02:47:57 UTC

[GitHub] beam pull request #4117: Add get_min_timestamp to Bundle

GitHub user nedgar opened a pull request:

    https://github.com/apache/beam/pull/4117

    Add get_min_timestamp to Bundle

    Improves performance by taking timestamps from _StackedWindowedValues
    rather than iterating over all elements.
    Memoizes min timestamp if bundle is committed.

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

    $ git pull https://github.com/nedgar/beam master

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

    https://github.com/apache/beam/pull/4117.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 #4117
    
----
commit 61ae92dcffa98203cbe8f56930802405de2524f0
Author: Nick Edgar <ni...@ca.ibm.com>
Date:   2017-11-13T02:47:35Z

    Add get_min_timestamp to Bundle
    
    Improves performance by taking timestamps from _StackedWindowedValues
    rather than iterating over all elements.
    Memoizes min timestamp if bundle is committed.

----


---