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

Rename ZkVersion to LongVersion #494

Closed
zhaijack opened this issue Sep 4, 2017 · 0 comments
Closed

Rename ZkVersion to LongVersion #494

zhaijack opened this issue Sep 4, 2017 · 0 comments

Comments

@zhaijack
Copy link
Contributor

zhaijack commented Sep 4, 2017

FEATURE REQUEST

  1. Please describe the feature you are requesting.

Since ZkVersion is a general class wrapping an integer/long based version number, it would be good to rename zkVersion to LongVersion and move it from metadata package to a versioning package.

  1. Indicate the importance of this issue to you (blocker, must-have, should-have, nice-to-have). Are you currently using any workarounds to address this issue?

nice-to-have

  1. Provide any additional detail on your proposed use case for this feature.

In #491, we are planning to introduce a metadata rpc service to wrap the existing ledger manager implementation. it would be good to support a integer/long based versioning, rather than a special case for zookeeper. so this is a change that is prepared for the upcoming changes in #491

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants