You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@beam.apache.org by jkff <gi...@git.apache.org> on 2017/03/03 20:01:44 UTC

[GitHub] beam pull request #2153: [BEAM-1422] ParDo should comply with PTransform sty...

GitHub user jkff opened a pull request:

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

    [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 https://github.com/jkff/incubator-beam pardo-style

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

    https://github.com/apache/beam/pull/2153.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 #2153
    
----
commit 11b3f133975f54f2a4f577b815e7a645aa007819
Author: Eugene Kirpichov <ki...@google.com>
Date:   2017-03-03T18:53:28Z

    Removes ParDo.Unbound and UnboundMulti

commit 1e2369389faffa10a927c4ee8d3d0bfbff171e86
Author: Eugene Kirpichov <ki...@google.com>
Date:   2017-03-03T18:53:59Z

    Removes accidentally left over class in test

commit 7f87e4413b97f982eb6590f6ad3aab8ba343e987
Author: Eugene Kirpichov <ki...@google.com>
Date:   2017-03-03T19:06:49Z

    Renamed ParDo.Bound to ParDo.SingleOutput

commit 596beb96d052d545932842956ae93ca15ce9199d
Author: Eugene Kirpichov <ki...@google.com>
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] beam pull request #2153: [BEAM-1422] ParDo should comply with PTransform sty...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

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


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---