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

Convert gossip's capacity key to store key #2239

Merged
merged 1 commit into from
Aug 24, 2015
Merged

Convert gossip's capacity key to store key #2239

merged 1 commit into from
Aug 24, 2015

Conversation

BramGruneir
Copy link
Member

We actually gossip the full store.description (which includes the capacity), so there's no reason to call it only capacity.

This also includes

  • Change the key suffix from nodeID-storeID to just storeID
  • Renaming of the gossip capacity functions to gossip store

@tamird
Copy link
Contributor

tamird commented Aug 24, 2015

Looks like the word "convert" snuck into the commit message one time too many. LGTM

@BramGruneir
Copy link
Member Author

It did indeed. Fixed.

@tamird
Copy link
Contributor

tamird commented Aug 24, 2015

You didn't change the commit message, though.

We actually gossip the full store.description (which includes the capacity), so there's no reason to call it only capacity.

This also includes
* Change the key suffix from nodeID-storeID to just storeID
* Renaming of the gossip capacity functions to gossip store.
@BramGruneir
Copy link
Member Author

You're too fast for me.

@tamird
Copy link
Contributor

tamird commented Aug 24, 2015

zoom

BramGruneir added a commit that referenced this pull request Aug 24, 2015
Convert gossip's `capacity` key to `store` key
@BramGruneir BramGruneir merged commit 67a29f1 into master Aug 24, 2015
@BramGruneir BramGruneir deleted the bram/gossip branch August 24, 2015 21:00
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