beam-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From jkff <>
Subject [GitHub] beam pull request #2153: [BEAM-1422] ParDo should comply with PTransform sty...
Date Fri, 03 Mar 2017 20:01:44 GMT
GitHub user jkff opened a pull request:

    [BEAM-1422] ParDo should comply with PTransform style guide

    Backward-incompatible changes:
    * Removes ParDo.Unbound and UnboundMulti (as a result, the only entry point is ParDo.of(DoFn)
- you can no longer specify ParDo.withSideInputs(...).of(fn) and such)
    * Renames ParDo.Bound to ParDo.SingleOutput and ParDo.BoundMulti to ParDo.MultiOutput
    R: @kennknowles 
    CC: @davorbonaci @dhalperi 

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

    $ git pull pardo-style

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 #2153
commit 11b3f133975f54f2a4f577b815e7a645aa007819
Author: Eugene Kirpichov <>
Date:   2017-03-03T18:53:28Z

    Removes ParDo.Unbound and UnboundMulti

commit 1e2369389faffa10a927c4ee8d3d0bfbff171e86
Author: Eugene Kirpichov <>
Date:   2017-03-03T18:53:59Z

    Removes accidentally left over class in test

commit 7f87e4413b97f982eb6590f6ad3aab8ba343e987
Author: Eugene Kirpichov <>
Date:   2017-03-03T19:06:49Z

    Renamed ParDo.Bound to ParDo.SingleOutput

commit 596beb96d052d545932842956ae93ca15ce9199d
Author: Eugene Kirpichov <>
Date:   2017-03-03T19:13:10Z

    Renamed ParDo.BoundMulti to ParDo.MultiOutput


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