Skip to content

Remove the DirectPipeline class - #182

Closed
tgroh wants to merge 1 commit into
apache:masterfrom
tgroh:remove_direct_pipeline
Closed

Remove the DirectPipeline class#182
tgroh wants to merge 1 commit into
apache:masterfrom
tgroh:remove_direct_pipeline

Conversation

@tgroh

@tgrohtgroh commented Apr 15, 2016

Copy link
Copy Markdown
Member

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

  • Make sure the PR title is formatted like:
    [BEAM-<Jira issue #>] Description of pull request
  • Make sure tests pass via mvn clean verify. (Even better, enable
    Travis-CI on your fork and ensure the whole test matrix passes).
  • Replace <Jira issue #> in the title with the actual Jira issue
    number, if there is one.
  • If this contribution is large, please file an Apache
    Individual Contributor License Agreement.

Users who wish to use the DirectPipelineRunner should do so by creating
a new Pipeline with the runner set in the PipelineOptions.

Replace existing uses with TestPipeline, and DataflowAssert if necessary.

Users who wish to use the DirectPipelineRunner should do so by creating
a new Pipeline with the runner set in the PipelineOptions.
@tgroh

Copy link
Copy Markdown
MemberAuthor

R: @lukecwik

@tgroh

Copy link
Copy Markdown
MemberAuthor

CC: @amitsela This changes the way in which TransformTranslatorTest is executed, but I believe has no semantic impact.

@lukecwik

Copy link
Copy Markdown
Member

LGTM

Probably should get rid of DataflowPipeline and all others like it.

@tgroh

Copy link
Copy Markdown
MemberAuthor

Yes. #186 to do that.

swegner pushed a commit to swegner/beam that referenced this pull request Apr 22, 2016
Add BundleFactory, ImmutabilityCheckingBundleFactory
iemejia pushed a commit to iemejia/beam that referenced this pull request Jan 12, 2018
mareksimunek pushed a commit to mareksimunek/beam that referenced this pull request May 9, 2018
…o-local-executor
173 rename inmem executor to local executor
damccorm pushed a commit that referenced this pull request Dec 19, 2022
Co-authored-by: Elias Segundo <elias.segundo@luisrazo.local>
Co-authored-by: Elias Segundo Antonio <eliassegundo.segundo@gmail.com>
Co-authored-by: Elias Segundo <elias.segundo@luisrazo.local>
lostluck pushed a commit to lostluck/beam that referenced this pull request Dec 22, 2022
Co-authored-by: Elias Segundo <elias.segundo@luisrazo.local>
Co-authored-by: Elias Segundo Antonio <eliassegundo.segundo@gmail.com>
Co-authored-by: Elias Segundo <elias.segundo@luisrazo.local>
pl04351820 pushed a commit to pl04351820/beam that referenced this pull request Dec 20, 2023
Co-authored-by: Christopher Wilcox <crwilcox@google.com>
Sign up for freeto 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.

2 participants

@tgroh@lukecwik