Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BEAM-1771] Clean up dataflow/google references/URLs in examples #2281

Merged
merged 1 commit into from
Mar 23, 2017

Conversation

melap
Copy link
Contributor

@melap melap commented Mar 21, 2017

R: @aaltay @davorbonaci

In examples/java/README, I separated and created a section for each runner so it's easy for other runners to add their commands also. Please verify I didn't mess up the command lines when doing that, as I'm not familiar with the bundling. Long term, I am wondering if most of the README content would be better off in the Word Count walkthrough on the website? as it's similar to what's there already, and then we can use tabs for Java/Python and the different runners. It'd also mean the user no longer needs to go back and forth between the walkthrough and the README.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 70.162% when pulling 81bcbb4 on melap:examples into f785584 on apache:master.

@asfbot
Copy link

asfbot commented Mar 21, 2017

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

Copy link
Member

@davorbonaci davorbonaci left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks @melap.

@asfgit asfgit merged commit 81bcbb4 into apache:master Mar 23, 2017
@melap melap deleted the examples branch March 23, 2017 20:44
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.

None yet

5 participants