Skip to content

Add example to BoundedCountingInput javadoc#2402

Merged
asfgit merged 1 commit intoapache:masterfrom
wtanaka:example
Apr 4, 2017
Merged

Add example to BoundedCountingInput javadoc#2402
asfgit merged 1 commit intoapache:masterfrom
wtanaka:example

Conversation

@wtanaka
Copy link
Contributor

@wtanaka wtanaka commented Apr 2, 2017

No description provided.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 70.136% when pulling 3345c2b on wtanaka:example into ea33e33 on apache:master.

@asfbot
Copy link

asfbot commented Apr 2, 2017

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/beam_PreCommit_Java_MavenInstall/9053/
--none--

* <pre>{@code
* Pipeline p = ...
* PTransform<PBegin, PCollection<Long>> producer = CountingInput.upTo(10L);
* PCollection<Long> bounded = p.apply(producer);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggest simplifying as:

PCollection<Long> bounded = p.apply(CountingInput.upTo(10L));

Usually, we don't save a PTransform in a local variable unless we want to reuse it or build it up over many lines of code – we normally just apply it inline.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

updated

@dhalperi
Copy link
Contributor

dhalperi commented Apr 3, 2017

R: @dhalperi

@wtanaka
Copy link
Contributor Author

wtanaka commented Apr 4, 2017

updated and rebased

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 65b03eb on wtanaka:example into ** on apache:master**.

@asfbot
Copy link

asfbot commented Apr 4, 2017

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/beam_PreCommit_Java_MavenInstall/9117/
--none--

@asfgit asfgit merged commit 65b03eb into apache:master Apr 4, 2017
asfgit pushed a commit that referenced this pull request Apr 4, 2017
@wtanaka wtanaka deleted the example branch April 14, 2017 06:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants