Skip to content

Commit

Permalink
add ZINTER command, added to Redis in 6.2
Browse files Browse the repository at this point in the history
  • Loading branch information
skipkayhil committed Jun 20, 2021
1 parent f759203 commit 68fe7b5
Show file tree
Hide file tree
Showing 5 changed files with 119 additions and 0 deletions.
39 changes: 39 additions & 0 deletions lib/redis.rb
Original file line number Diff line number Diff line change
Expand Up @@ -2062,6 +2062,45 @@ def zcount(key, min, max)
end
end

# Return the intersection of multiple sorted sets
#
# @example Retrieve the intersection of `2*zsetA` and `1*zsetB`
# redis.zinter("zsetA", "zsetB", :weights => [2.0, 1.0])
# # => ["v1", "v2"]
# @example Retrieve the intersection of `2*zsetA` and `1*zsetB`, and their scores
# redis.zinter("zsetA", "zsetB", :weights => [2.0, 1.0], :with_scores => true)
# # => [["v1", 3.0], ["v2", 6.0]]
#
# @param [String, Array<String>] keys one or more keys to intersect
# @param [Hash] options
# - `:weights => [Float, Float, ...]`: weights to associate with source
# sorted sets
# - `:aggregate => String`: aggregate function to use (sum, min, max, ...)
# - `:with_scores => true`: include scores in output
#
# @return [Array<String>, Array<[String, Float]>]
# - when `:with_scores` is not specified, an array of members
# - when `:with_scores` is specified, an array with `[member, score]` pairs
def zinter(*keys, weights: nil, aggregate: nil, with_scores: false)
args = [:zinter, keys.size, *keys]

if weights
args << "WEIGHTS"
args.concat(weights)
end

args << "AGGREGATE" << aggregate if aggregate

if with_scores
args << "WITHSCORES"
block = FloatifyPairs
end

synchronize do |client|
client.call(args, &block)
end
end

# Intersect multiple sorted sets and store the resulting sorted set in a new
# key.
#
Expand Down
7 changes: 7 additions & 0 deletions lib/redis/distributed.rb
Original file line number Diff line number Diff line change
Expand Up @@ -674,6 +674,13 @@ def zcount(key, min, max)
node_for(key).zcount(key, min, max)
end

# Get the intersection of multiple sorted sets
def zinter(*keys, **options)
ensure_same_node(:zinter, keys) do |node|
node.zinter(*keys, **options)
end
end

# Intersect multiple sorted sets and store the resulting sorted set in a new
# key.
def zinterstore(destination, keys, **options)
Expand Down
12 changes: 12 additions & 0 deletions test/cluster_commands_on_sorted_sets_test.rb
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,18 @@ class TestClusterCommandsOnSortedSets < Minitest::Test
include Helper::Cluster
include Lint::SortedSets

def test_zinter
assert_raises(Redis::CommandError) { super }
end

def test_zinter_with_aggregate
assert_raises(Redis::CommandError) { super }
end

def test_zinter_with_weights
assert_raises(Redis::CommandError) { super }
end

def test_zinterstore
assert_raises(Redis::CommandError) { super }
end
Expand Down
12 changes: 12 additions & 0 deletions test/distributed_commands_on_sorted_sets_test.rb
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,18 @@ class TestDistributedCommandsOnSortedSets < Minitest::Test
include Helper::Distributed
include Lint::SortedSets

def test_zinter
assert_raises(Redis::Distributed::CannotDistribute) { super }
end

def test_zinter_with_aggregate
assert_raises(Redis::Distributed::CannotDistribute) { super }
end

def test_zinter_with_weights
assert_raises(Redis::Distributed::CannotDistribute) { super }
end

def test_zinterstore
assert_raises(Redis::Distributed::CannotDistribute) { super }
end
Expand Down
49 changes: 49 additions & 0 deletions test/lint/sorted_sets.rb
Original file line number Diff line number Diff line change
Expand Up @@ -432,6 +432,55 @@ def test_zunionstore_expand
assert_equal 5, r.zunionstore('{1}baz', %w[{1}foo {1}bar])
end

def test_zinter
target_version("6.2") do
r.zadd 'foo', 1, 's1'
r.zadd 'bar', 2, 's1'
r.zadd 'foo', 3, 's3'
r.zadd 'bar', 4, 's4'

assert_equal ['s1'], r.zinter('foo', 'bar')
assert_equal [['s1', 3.0]], r.zinter('foo', 'bar', with_scores: true)
end
end

def test_zinter_with_weights
target_version("6.2") do
r.zadd 'foo', 1, 's1'
r.zadd 'foo', 2, 's2'
r.zadd 'foo', 3, 's3'
r.zadd 'bar', 20, 's2'
r.zadd 'bar', 30, 's3'
r.zadd 'bar', 40, 's4'

assert_equal %w[s2 s3], r.zinter('foo', 'bar')
assert_equal [['s2', 22.0], ['s3', 33.0]], r.zinter('foo', 'bar', with_scores: true)

assert_equal %w[s2 s3], r.zinter('foo', 'bar', weights: [10, 1])
assert_equal [['s2', 40.0], ['s3', 60.0]], r.zinter('foo', 'bar', weights: [10, 1], with_scores: true)
end
end

def test_zinter_with_aggregate
target_version("6.2") do
r.zadd 'foo', 1, 's1'
r.zadd 'foo', 2, 's2'
r.zadd 'foo', 3, 's3'
r.zadd 'bar', 20, 's2'
r.zadd 'bar', 30, 's3'
r.zadd 'bar', 40, 's4'

assert_equal %w[s2 s3], r.zinter('foo', 'bar')
assert_equal [['s2', 22.0], ['s3', 33.0]], r.zinter('foo', 'bar', with_scores: true)

assert_equal %w[s2 s3], r.zinter('foo', 'bar', aggregate: :min)
assert_equal [['s2', 2.0], ['s3', 3.0]], r.zinter('foo', 'bar', aggregate: :min, with_scores: true)

assert_equal %w[s2 s3], r.zinter('foo', 'bar', aggregate: :max)
assert_equal [['s2', 20.0], ['s3', 30.0]], r.zinter('foo', 'bar', aggregate: :max, with_scores: true)
end
end

def test_zinterstore
r.zadd 'foo', 1, 's1'
r.zadd 'bar', 2, 's1'
Expand Down

7 comments on commit 68fe7b5

@alan-sapaad
Copy link

Choose a reason for hiding this comment

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

When will this code be released, @skipkayhil ?

@skipkayhil
Copy link
Contributor Author

Choose a reason for hiding this comment

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

When will this code be released, @skipkayhil ?

I'm not a member of the repo, so you would have to ask someone like @byroot

You can use the github version in your Gemfile if you need it sooner (I've been doing that here):
https://github.com/SkipKayhil/friday/blob/44b26d2fb610947ba903177dbfc2a7d50bb1ca45/Gemfile#L15

@alan-sapaad
Copy link

@alan-sapaad alan-sapaad commented on 68fe7b5 Jul 22, 2021

Choose a reason for hiding this comment

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

@skipkayhil I tried doing that by
gem 'redis', :git => "git://github.com/redis/redis-rb.git", :branch => "master"
But I am still not able to access zinter, redis is throwing missing method error

Sidekiq was also throwing an error like "not able to find redis >=4.2". I have the setup inside a docker
Any idea why this might be happening?

@skipkayhil
Copy link
Contributor Author

Choose a reason for hiding this comment

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

It sounds like your redis verion may not be 6.2

You can check with the info hash: Redis.current.info['redis_version']

@alan-sapaad
Copy link

@alan-sapaad alan-sapaad commented on 68fe7b5 Jul 23, 2021

Choose a reason for hiding this comment

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

Oh you're right. Thanks @skipkayhil.
I had upgraded the Redis version in my docker-compose file. But I had to do docker-compose up --force-recreate to actually spin up the upgraded redis instance.
One more question though; If I want to use this zinter command, would it be safer to just point my redis gem to the master branch, or this specific commit, until this is released under a tag?

@alan-sapaad
Copy link

Choose a reason for hiding this comment

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

And also, I just noticed, my Sidekiq now cannot find the Redis GEM.
It is throwing

Gem::MissingSpecError:
  Could not find 'redis' (>= 4.2.0) among 153 total gem(s)

But I can access Redis through by rails console. Is there anything I have to do when I am pulling a gem from Github?

@alan-sapaad
Copy link

Choose a reason for hiding this comment

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

Ok, got it. I had to do bundle exec. Thanks!

Please sign in to comment.