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

Follow up ioredis client switch #537

Open
2 of 5 tasks
XVincentX opened this issue Nov 15, 2017 · 1 comment
Open
2 of 5 tasks

Follow up ioredis client switch #537

XVincentX opened this issue Nov 15, 2017 · 1 comment

Comments

@XVincentX
Copy link
Member

XVincentX commented Nov 15, 2017

Interesting followups:

ioredis

  • Take advantage of transparent key prefixing so we can remove some boilerplate code

  • Scan commands can return a stream — it is a great idea in order to avoid killing the server (both redis and the gateway)

  • Cluster support. This is a different connection object with a different set of connection options as well.

ioredis-mock

  • Re—enable the db.test.js test that was checking certificate files. That should not be that hard.

  • Continuously track ioredis-mock for argument/reply transformer support here: hmset should allow accepting an object stipsan/ioredis-mock#322. We cut a corner to make it happen and we should repay the debt once the main library will be fixed.

@stipsan
Copy link

stipsan commented Nov 27, 2017

New release with transformers support just got pushed https://github.com/stipsan/ioredis-mock/releases/tag/v3.4.0

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

3 participants