M Sazzadul Hoque
M Sazzadul Hoque
@dengliming For the particular Params class that you mentioned, yes, your approach is better. But keeping in a Map allowed all Params class to have a _base_ class. However, many...
@Razi007 as a new contributor, I think you can start with [Redis 7 commands to implement](https://github.com/redis/jedis/discussions/2807).
@Razi007 To get some ideas, you can check PRs that are already merged. E.g. about implementing _regular_ commands, you can check #2843 which is pretty straightforward. #2771 is a bit...
@Razi007 Sure.
@SeaSand1024 Have you checked if this implementation is actually compatible with RedisShardAPI?
Ref #2275: I don't completely support throwing exception because we don't do that even from Jedis class. Throwing exception from JedisCluster constructor seems contradictory to me.
According to current Jedis structure, we can't add this as a method because all of the subsequent command executions will fail. It can used internally when we introduce any fire-n-forget...
Yes, the sleep time is not configurable. We welcome a PR. @barbasa Apologies for late reply.
Dupe #3042
@aarengee Thank you for descriptive feature request. Keeping it here for discussion.