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

Fixed SearchKeys method returning keys with KeyPrefix included #156

Merged
merged 1 commit into from
Nov 27, 2018

Conversation

carlosboeing
Copy link
Contributor

@carlosboeing carlosboeing commented Nov 17, 2018

I ran some tests to make sure the string manipulation on the keys list won't add too much overhead to the method.

Here are the results:

100K records:
with prefix 326ms
without prefix 305ms

500K records:
with prefix 1620ms
without prefix 1589ms

1M records:
with prefix 3176ms
without prefix 3139ms

@imperugo
Copy link
Owner

Thanks.
I'm going to review, merge and publish it tomorrow.

@imperugo imperugo merged commit ea173a9 into imperugo:master Nov 27, 2018
@imperugo
Copy link
Owner

On nuget now.

Thanks again

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