Skip to content

added support for bitfield command. #178

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

Merged
merged 2 commits into from
Dec 15, 2022
Merged

Conversation

slice4e
Copy link
Collaborator

@slice4e slice4e commented Dec 15, 2022

added support for bitfield command.

@filipecosta90
Copy link
Contributor

 2022-12-15 20:20:27 WARNING there is a difference between specified test-commands in the yaml (name=memtier_benchmark-1key-100MB-string-bitfield) and the ones we've detected ['get']!=['get', 'bitfield']
2022-12-15 20:20:27 WARNING there is a difference between specified test-groups in the yaml (name=memtier_benchmark-1key-100MB-string-bitfield) and the ones we've detected ['string']!=['bitmap']

@slice4e please correct the groups of test :)

@filipecosta90 filipecosta90 self-requested a review December 15, 2022 20:24
Copy link
Contributor

@filipecosta90 filipecosta90 left a comment

Choose a reason for hiding this comment

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

2022-12-15 20:20:27 WARNING there is a difference between specified test-commands in the yaml (name=memtier_benchmark-1key-100MB-string-bitfield) and the ones we've detected ['get']!=['get', 'bitfield']
2022-12-15 20:20:27 WARNING there is a difference between specified test-groups in the yaml (name=memtier_benchmark-1key-100MB-string-bitfield) and the ones we've detected ['string']!=['bitmap']

@filipecosta90 filipecosta90 added the spec Improvements or additions to the Redis benchmarks specification label Dec 15, 2022
@filipecosta90 filipecosta90 self-requested a review December 15, 2022 20:34
@slice4e
Copy link
Collaborator Author

slice4e commented Dec 15, 2022

I believe I fixed it. and pushed a new commit

@filipecosta90
Copy link
Contributor

I believe I fixed it. and pushed a new commit

Confirmed fixed in https://github.com/redis/redis-benchmarks-specification/actions/runs/3707733491/jobs/6284499636
Merging. Thank you @slice4e !

@filipecosta90 filipecosta90 merged commit b702dd9 into redis:main Dec 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
spec Improvements or additions to the Redis benchmarks specification
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants