Need support for `client.duplicate()` about handy-redis HOT 5 CLOSED

Hmm I'm not sure I understand the use-case. Do you have a code example of how you'd like to handle this situation if the library supported it?

from handy-redis.

It would be like:

const blockingClient = client.duplicate();

instead of createHandyClient(await client.duplicate()). Thanks.

from handy-redis.

Do you know if client.duplicate() is always synchronous? I'm wary to put this in, in case that's not always the case. I haven't used it myself, and I assume it's an async call in node_redis for a reason.

In the meantime, if it's definitely safe in your case, could you avoid the await (which I can see is annoying if you want to create a client in a non-async context) by doing

const blockingClient = createHandyClient(client.duplicate() as any);

?

from handy-redis.

I didn't know if that workaround would work because client.duplicate() with handy-redis returns a Promise. Well, then I suggest you make a high-level API that could be a shortcut of createHandyRedis(client.duplicate()), like client.duplicateAsyncClient() or something.

from handy-redis.

You're right - it would be a promise. I would still rather not add this to the API, I'd like to avoid special cases where possible. This is working for me though:

it("can use duplicate", async () => {
    const first = createHandyClient();
    const second = createHandyClient(first.redis.duplicate());

    await first.set("one", "hello");
    await second.set("two", "goodbye");

    expect(await first.get("two")).toEqual("goodbye");
    expect(await second.get("one")).toEqual("hello");
});

This should be good enough, unless I can find a better way to handle synchronous node_redis methods.

from handy-redis.