-
Notifications
You must be signed in to change notification settings - Fork 60
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 KV Filtering of keys #545
Conversation
… Same as V1 client.
…fo such as -preview. Also changed name to .Net to .NET for the client name.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM thanks @darkwatchuk
@mtmk , how important is this bit from the ADR? It looks as though it doesn't impact this implementation. Just making sure we don't need an additional check.
|
@rickdotnet good point. created an issue for it: |
Resolves #521