-
Notifications
You must be signed in to change notification settings - Fork 532
Change BloomFilter implementation for Sparse Joins #1806
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
Merged
Merged
Changes from 37 commits
Commits
Show all changes
46 commits
Select commit
Hold shift + click to select a range
d70edee
update docs
nevillelyh 8181a11
fix maven-metadata.xml URL in scripts/bump_scio.sh
nevillelyh b4c2a0c
Move scio-contrib back to scio-extra (#1776)
nevillelyh 9584792
Add a hand written Coder for pairs (#1775)
jto b75aaa1
Make BQ annotations serializable (#1773)
anish749 d3a41b5
use golang image for CircleCI deploy, fix #1772 (#1779)
nevillelyh 01dd7bc
Remove the usage of Future around ScioContext and Tap's (#1666)
regadas e5675b9
Fix benchmark rebase
regadas 62bbd20
Add schema and row coders (#1698)
jto dc6f4a4
Simplify query row transform (#1767)
regadas f48c2cd
Fix typos
jto 4a8496c
use camelCase for typed arguments, fix #1770 (#1780)
nevillelyh ddf1f09
Rework sql syntax (#1778)
regadas ac346dc
Fix: use same protoc (#1781)
regadas 65f0f43
Bump coursier version (#1782)
regadas 6bb54ed
Merge remote-tracking branch 'origin/master'
anish749 a8fba4f
update pair scollection functions to use mutable bloom filters.
anish749 96b9088
turn off scalastyle for use of return
anish749 80ab0b5
Merge branch 'master' of github.com:spotify/scio
anish749 def347f
Merge branch 'master' of github.com:spotify/scio
anish749 244210a
Merge branch 'master' into mutableBloomFilters
anish749 5ebb747
add sparse bloom filter implementation to save memory
anish749 4d63959
delayed mutable bf
756d6f4
fix bf set init condition, delayed init tests.
5780fae
clean up sparse mutable bf
32348f5
less values
1a59d10
add gens for sparse mutable bf
c3e0fbd
Merge branch 'master2k19' into mutableBloomFilters
anish749 3f05ce6
new scala fmt
anish749 dcb8056
fix imports and ret type
anish749 0f251dd
Update scio-core/src/main/scala/com/spotify/scio/util/BloomFilter.scala
nevillelyh d416e6a
Update scio-core/src/main/scala/com/spotify/scio/util/BloomFilter.scala
nevillelyh d346d61
Update scio-core/src/main/scala/com/spotify/scio/util/BloomFilter.scala
nevillelyh 070bb8b
Update scio-core/src/main/scala/com/spotify/scio/util/BloomFilter.scala
nevillelyh a5fed50
Update scio-core/src/main/scala/com/spotify/scio/util/BloomFilter.scala
nevillelyh 1b719c3
Update scio-core/src/main/scala/com/spotify/scio/util/BloomFilter.scala
nevillelyh cdfe002
Update scio-core/src/main/scala/com/spotify/scio/util/BloomFilter.scala
nevillelyh 699fe13
add mutation detection tests
anish749 4f1a3d7
Merge branch 'master' into mutableBloomFilters
52f6313
Update scio-core/src/main/scala/com/spotify/scio/util/BloomFilter.scala
regadas ae146cf
Apply suggestions from code review
regadas 9ff97a7
address review comments
anish749 e6bab9f
more restricted access
anish749 66bd624
foreach -> while
anish749 5ed4707
add warning
anish749 2ff5176
remove return
anish749 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.