Skip to content

Replaces sources/sinks in programming guide with IOs#386

Merged
asfgit merged 1 commit intoasf-sitefrom
no-sinks
Feb 14, 2018
Merged

Replaces sources/sinks in programming guide with IOs#386
asfgit merged 1 commit intoasf-sitefrom
no-sinks

Conversation

@jkff
Copy link

@jkff jkff commented Feb 7, 2018

The Sink API no longer exists. The Source API is a very advanced API that has no place in a high-level programming guide. The proper way for users to think about reading/writing data is to use IOs that come with Beam.

The Sink API no longer exists. The Source API is a very advanced API that has no place in a high-level programming guide. The proper way for users to think about reading/writing data is to use IOs that come with Beam.
@jkff jkff requested a review from melap February 7, 2018 20:50
@jbonofre jbonofre self-requested a review February 7, 2018 20:54
Copy link

@melap melap left a comment

Choose a reason for hiding this comment

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

LGTM

@jkff
Copy link
Author

jkff commented Feb 10, 2018

@asfgit merge

@asfgit
Copy link

asfgit commented Feb 10, 2018

Error: Couldn't complete force push to verification branch. Please try again.

@jkff
Copy link
Author

jkff commented Feb 13, 2018

@asfgit merge

@asfgit
Copy link

asfgit commented Feb 13, 2018

Error: Couldn't complete force push to verification branch. Please try again.

@jkff
Copy link
Author

jkff commented Feb 14, 2018

@asfgit merge

asfgit pushed a commit that referenced this pull request Feb 14, 2018
@asfgit
Copy link

asfgit commented Feb 14, 2018

MergeBot: Merge: PR 386 in state failure with description Timed out trying to find verification job. Check Jenkins (https://builds.apache.org/job/beam_PreCommit_Website_Merge/) to ensure job is configured correctly.. URL: http://mergebot-vm2.apache.org/beam-website/386.

@asfgit
Copy link

asfgit commented Feb 14, 2018

Error: Timed out trying to find verification job. Check Jenkins (https://builds.apache.org/job/beam_PreCommit_Website_Merge/) to ensure job is configured correctly.

@melap
Copy link

melap commented Feb 14, 2018

@asfgit merge

asfgit pushed a commit that referenced this pull request Feb 14, 2018
@asfgit asfgit merged commit a0bc5ff into asf-site Feb 14, 2018
@jkff jkff deleted the no-sinks branch February 14, 2018 06:41
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.

3 participants