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-825] Fill in the documentation/runners/apex portion of the website #219

Closed
wants to merge 3 commits into from

Conversation

tweise
Copy link

@tweise tweise commented Apr 21, 2017

R: @davorbonaci

This is on top of #98

Please note that the example command doesn't actually use HDFS yet, the provider isn't present.

@asfbot
Copy link

asfbot commented Apr 21, 2017

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/beam_PreCommit_Website_Stage/426/

Jenkins built the site at commit id b681fa2 with Jekyll and staged it here. Happy reviewing.

Note that any previous site has been deleted. This staged site will be automatically deleted after its TTL expires. Push any commit to the pull request branch or re-trigger the build to get it staged again.

@asfbot
Copy link

asfbot commented Apr 21, 2017

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/beam_PreCommit_Website_Stage/428/

Jenkins built the site at commit id 239e000 with Jekyll and staged it here. Happy reviewing.

Note that any previous site has been deleted. This staged site will be automatically deleted after its TTL expires. Push any commit to the pull request branch or re-trigger the build to get it staged again.

@asfbot
Copy link

asfbot commented Apr 21, 2017

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/beam_PreCommit_Website_Stage/429/

Jenkins built the site at commit id 7e9aa08 with Jekyll and staged it here. Happy reviewing.

Note that any previous site has been deleted. This staged site will be automatically deleted after its TTL expires. Push any commit to the pull request branch or re-trigger the build to get it staged again.

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. Merging.

(Please keep the bug open to track the removal of the comment that says to change the underlying example.)

@asfgit asfgit closed this in 35d6306 Apr 21, 2017
@tweise
Copy link
Author

tweise commented Apr 22, 2017

@davorbonaci I created BEAM-2053 - please comment if the suggested change to the examples makes sense.

@davorbonaci
Copy link
Member

(makes sense)

robertwb pushed a commit to robertwb/incubator-beam that referenced this pull request Jun 5, 2018
melap pushed a commit to apache/beam that referenced this pull request Jun 20, 2018
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

3 participants