sonic-swss-common icon indicating copy to clipboard operation
sonic-swss-common copied to clipboard

Published 20 hours ago • verified publisher icon sonic-net

Add REDIS async support

Open martin-belanger opened this issue 4 years ago • 3 comments

This pull request is mainly to add support for REDIS asynchronous APIs (from the hiredis library).

I also fixed a potential memory leak in common/rediscommand.cpp.

martin-belanger avatar Oct 06 '20 19:10 martin-belanger

can you separate the bug fix from this pr? we can review and merge it separately.

lguohan avatar Oct 07 '20 05:10 lguohan

@qiluo-msft, any update on this pr?

lguohan avatar Nov 26 '20 11:11 lguohan

I just added a new commit to address the syntax issues (camel-casing, blanks, etc.)

I will look into adding a class RedisAsyncContext derived from RedisContext when I find the time or I will find someone else to do that.

martin-belanger avatar Dec 04 '20 20:12 martin-belanger