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-95] Use snappy compression on objects serialized via SerializabUtils #22

Closed
wants to merge 2 commits into from

Conversation

lukecwik
Copy link
Member

@lukecwik lukecwik commented Mar 4, 2016

@lukecwik
Copy link
Member Author

lukecwik commented Mar 4, 2016

R: @dhalperi

@lukecwik
Copy link
Member Author

lukecwik commented Mar 4, 2016

R: @kennknowles

@dhalperi
Copy link
Contributor

dhalperi commented Mar 4, 2016

LGTM

@asfgit asfgit closed this in 9be2944 Mar 4, 2016
davorbonaci added a commit to GoogleCloudPlatform/DataflowJavaSDK that referenced this pull request Mar 8, 2016
@lukecwik lukecwik deleted the snappy branch March 25, 2016 18:46
echauchot added a commit to echauchot/beam that referenced this pull request May 12, 2017
Workaround for issue apache#22 + extra cleaning
echauchot added a commit to echauchot/beam that referenced this pull request May 12, 2017
cosmoskitten pushed a commit to cosmoskitten/beam that referenced this pull request Jun 16, 2017
Workaround for issue apache#22 + extra cleaning

Replace junit asserts by hamcrest asserts
Set numEvents in test to the minimum number that makes the tests pass
issue apache#15

comments, improve asserts (hamcrest), reformat

For now make generate monothreaded
cosmoskitten pushed a commit to cosmoskitten/beam that referenced this pull request Jun 16, 2017
Fix Runner categories in tests

Add streaming unit tests and corresponding labels
issue apache#37

Update numEvents: results are no more linked to the number of events

issue apache#22
asfgit pushed a commit that referenced this pull request Aug 23, 2017
Workaround for issue #22 + extra cleaning

Replace junit asserts by hamcrest asserts
Set numEvents in test to the minimum number that makes the tests pass
issue #15

comments, improve asserts (hamcrest), reformat

For now make generate monothreaded
asfgit pushed a commit that referenced this pull request Aug 23, 2017
Fix Runner categories in tests

Add streaming unit tests and corresponding labels
issue #37

Update numEvents: results are no more linked to the number of events

issue #22
mareksimunek pushed a commit to mareksimunek/beam that referenced this pull request May 9, 2018
#! Mention maven dependencies in README.md
tvalentyn pushed a commit to tvalentyn/beam that referenced this pull request May 15, 2018
robertwb pushed a commit to robertwb/incubator-beam that referenced this pull request Apr 30, 2020
* Migrate github_sample functionality from Jekyll to Hugo

* fixup! Migrate github_sample functionality from Jekyll to Hugo
hengfengli referenced this pull request in hengfengli/beam Mar 21, 2022
When we are splitting a wait for child partitions or a wait for parent
partitions we need to make sure the restriction residual does not skip a
phase, when scheduled before the primary. This is problematic in the
case where the primary resumes, such as in the wait for child partitions
/ wait for parent partitions.
In this PR, when splitting one of the two aforementioned modes, we
remain at the same mode, instead of advancing. This will make sure the
residual only proceeds if the current phase is complete.
sjvanrossum pushed a commit to sjvanrossum/beam that referenced this pull request May 17, 2023
…methods-from-PValue

refactor: remove PValue::register_pipeline_*() methods  …
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