duplicate() is missing on RedisSentinelType

[CSenshi]

Description

RedisClientType and RedisClusterType both have .duplicate() method.
Should RedisSentinelType not implement this?

Node.js Version

No response

Redis Server Version

No response

Node Redis Version

No response

Platform

No response

Logs

[nkaradzhov]

@CSenshi thanks for raising this! On first glance I dont see any reason why not to have the duplicate method. In my eyes this is a low priority though. Not sure when I will have time to look at it

[elimelt]

I think having parity between the base client and sentinel client is a huge devex win, especially for those migrating to sentinel. Considering how similar a single leader vs leader + replica(s) setup is, sentinel could largely be a drop-in replacement from a client perspective, of course with caveats in case of failures.

@nkaradzhov do you mind taking a look if I open a PR for this?