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

Add incrby to zcache #72

Merged
merged 1 commit into from
Apr 13, 2015
Merged

Add incrby to zcache #72

merged 1 commit into from
Apr 13, 2015

Conversation

chaosgame
Copy link
Contributor

Hello @xiao, @nicks,

Please review the following commits I made in branch 'nathan-IncrBy'.

2ebb028 (2015-03-13 11:38:50 -0700)
Add incrby to zcache

R=@xiao
R=@nicks

@nicks
Copy link
Contributor

nicks commented Mar 13, 2015

lgtm

* Increment a key.
*
* @param {string} key The key to set
* @param {number} increment The value to increment by. 1 if unspecified.
Copy link
Contributor

Choose a reason for hiding this comment

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

{number=}

@nicks
Copy link
Contributor

nicks commented Apr 9, 2015

what's the status of this pr?

@chaosgame
Copy link
Contributor Author

This was added for supporting event de-duplication. I'm happy to commit it or abandon it, but it's not blocking anything anymore.

@chaosgame
Copy link
Contributor Author

Alright, I'm going to bump the version number and commit this for some of the work that @andrew3886 is doing.

chaosgame added a commit that referenced this pull request Apr 13, 2015
@chaosgame chaosgame merged commit c73ee28 into master Apr 13, 2015
@chaosgame chaosgame deleted the nathan-IncrBy branch April 13, 2015 19:03
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.

3 participants