Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Hiredis: Advanced async-handling #983

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Commits on Oct 3, 2021

  1. Hiredis: Advanced async-handling

    + hiredis.c: Convert redisSetError to take format and va_args
    + dict.*   : Fixed generic hash function -> cannot really take signed number
    + async.*  : Valdup function now handles OOM properly
                 KeyCompare function uses proper sized integers
                 nextArgument - change to use proper str-int conversion function
                 Added redisAsyncAppend* function family to support buffer
                                                     optimizations in async-mode
                 Added proper error handling on OOM errors [inc buffer rollback]
    
    Review fixes
    
    + hiredis.c : Reverted back to memcpy(), will test *printf in different time
    + aysnc.c   : Minor changes for better readability
    
    Handle unlikely/pathological OOM conditon cleanly.
    
    More OOM checks
    oshamash committed Oct 3, 2021
    Configuration menu
    Copy the full SHA
    44f434c View commit details
    Browse the repository at this point in the history