wangjia184/sortedset

Add Remove Option in `GetByScoreRangeOptions`

Opened this issue · 5 comments

This is a feature request to add a remove option to the options struct for GetByScoreRange. I have a branch and test case written, @wangjia184 could I have write access to push my branch and open a PR?

The use case is to "Pop" a range of scores without another iteration using Remove. I would like to use sortedset to delay values to a unix timestamp in the future(now + buffer). Then use GetByScoreRange to retrieve the items that are in the past like a queue. It will allow me to buffer messages and agg duplicates by key before publishing them.

Hello @parkerroan
PR is welcome, could you please raise an PR?

@wangjia184 please invite me as a collaborator and I will push the branch up and open a PR. I don't think I am able to without that.

ERROR: Permission to wangjia184/sortedset.git denied to parkerroan.
fatal: Could not read from remote repository.

@parkerroan you need create your own repository by folking this one, then apply changes in your branch, and raise a PR from your branch to this one. Everyone can create a PR from their own branch.

Sorry about that. Just opened the PR