Skip to content

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
@slice4e slice4e deleted the add_bitfield branch October 15, 2025 18:35
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