[GH-ISSUE #45] Change package API to hide redis client library used internally #1027

Closed
opened 2026-03-07 22:04:12 +03:00 by kerem · 0 comments
Owner

Originally created by @hibiken on GitHub (Jan 7, 2020).
Original GitHub issue: https://github.com/hibiken/asynq/issues/45

Originally assigned to: @hibiken on GitHub.

Is your feature request related to a problem? Please describe.
Currently, both Client and Background constructor takes go-redis/redis/v7 as a parameter. This will become an issue when there is a new version or if we want to switch to another redis-client library. The change will require major version update, which we don't want to do just for a redis-client library change.

Describe the solution you'd like
We should abstract this away from asynq package users and take RedisClientOption as a parameter.

Describe alternatives you've considered
None so far.

Originally created by @hibiken on GitHub (Jan 7, 2020). Original GitHub issue: https://github.com/hibiken/asynq/issues/45 Originally assigned to: @hibiken on GitHub. **Is your feature request related to a problem? Please describe.** Currently, both `Client` and `Background` constructor takes `go-redis/redis/v7` as a parameter. This will become an issue when there is a new version or if we want to switch to another redis-client library. The change will require major version update, which we don't want to do just for a redis-client library change. **Describe the solution you'd like** We should abstract this away from `asynq` package users and take `RedisClientOption` as a parameter. **Describe alternatives you've considered** None so far.
kerem 2026-03-07 22:04:12 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/asynq#1027
No description provided.