beam-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From amitsela <>
Subject [GitHub] incubator-beam pull request: [BEAM-189] The Spark runner uses valu...
Date Thu, 14 Apr 2016 21:01:15 GMT
GitHub user amitsela opened a pull request:

    [BEAM-189] The Spark runner uses valueInEmptyWindow which causes values to be dropped

    Be sure to do all of the following to help us incorporate your contribution
    quickly and easily:
     - [ ] Make sure the PR title is formatted like:
       `[BEAM-<Jira issue #>] Description of pull request`
     - [ ] Make sure tests pass via `mvn clean verify`. (Even better, enable
           Travis-CI on your fork and ensure the whole test matrix passes).
     - [ ] Replace `<Jira issue #>` in the title with the actual Jira issue
           number, if there is one.
     - [ ] If this contribution is large, please file an Apache
           [Individual Contributor License Agreement](

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

    $ git pull BEAM-189

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

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

    This closes #179
commit d9ce6df3c41dcbc7d884d5b19841475fac330757
Author: Sela <>
Date:   2016-04-14T19:01:15Z

    Replace valueInEmptyWindows with valueInGlobalWindow

commit 2309ad6dbd74333808d6b993b43fe81791a19611
Author: Sela <>
Date:   2016-04-14T19:02:20Z

    Replace valueInEmptyWindows with valueInGlobalWindow in Spark Function, and add per-value
    windowing functions

commit 47fdf133da440a9ee8a6a1bc5ea34e59138f7c53
Author: Sela <>
Date:   2016-04-14T19:05:14Z

    Materialize PCollection/RDD as windowed values with the appropriate windows.

commit 06afe790c3e933f7a1a5ed26c3586c22dbaf750f
Author: Sela <>
Date:   2016-04-14T20:18:24Z

    Add unit test for TextIO output to support the mvn exec:exec example we provide in README

commit 51089e52474127e4dba21014ca79e1e8879b98cd
Author: Sela <>
Date:   2016-04-14T20:58:14Z

    Satisfy checkstyle


If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at or file a JIRA ticket
with INFRA.

View raw message