Skip to content

Conversation

@tgroh
Copy link
Member

@tgroh tgroh commented Apr 14, 2016

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.

This PTransform Reifies and gathers all panes produced for each
window, outputting a single pane per window at the time the window is
Garbage Collected.

For use in PAssert, to match over the final contents of each window.

This PTransform Reifies and gathers all panes produced for each
window, outputting a single pane per window at the time the window is
Garbage Collected.

For use in PAssert, to match over the final contents of each window.
@tgroh tgroh force-pushed the accumulate_all_panes branch from 5d4ae9e to 3cda11a Compare April 14, 2016 22:58
@tgroh tgroh changed the title [BEAM-155] Add AccumulateAllPanes PTransform [BEAM-155] Add GatherAllPanes PTransform Apr 14, 2016
import org.apache.beam.sdk.values.TypeDescriptor;

/**
* Accumulates all panes that are output to the input {@link PCollection}, reifying the contents of
Copy link
Contributor

Choose a reason for hiding this comment

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

/**
 * Gathers all panes of each window into exactly one output.
 * <p>
 * Note that this will delay the output of a window until the garbage collection
 * time (when the watermark passes the end of the window plus allowed lateness) even if the up-stream triggers closed the window earlier.
 */

Copy link
Member Author

Choose a reason for hiding this comment

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

Done.

@bjchambers
Copy link
Contributor

LGTM

@asfgit asfgit closed this in d440d94 Apr 15, 2016
swegner pushed a commit to swegner/beam that referenced this pull request Apr 22, 2016
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
…y-flow

157 enable outputs to modify flow
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>
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.

2 participants