neophenix / Redis-hiredis

Perl wrapper around hiredis client
BSD 3-Clause "New" or "Revised" License
11 stars 8 forks source link

Using Redis::hiredis with more than 1024 opened file descriptors leads to infinite wait on select call #3

Closed biancalana closed 12 years ago

biancalana commented 12 years ago

I´m using Redis::hiredis inside a mod_perl module on an Apache with more than 1000 domains.

That leads the program to have more than 1024 opened file descriptors what trigger a bug in net.c where redisContextWaitReady() call select with FD_SETSIZE, that's generally 1024 in most operating systems, making select call never return.

That was corrected less than a month ago in https://github.com/antirez/hiredis/commits/master/net.c substituting sellect by poll call.

I've done a test updating the entire net.c with lastest from antirez repo and the problem does not happen anymore.

Can you update the module ?

Regards, Alexandre

neophenix commented 12 years ago

Thanks, updated net.c in 4f5f68a

biancalana commented 12 years ago

Was CPAN updated too ?

neophenix commented 12 years ago

It was not, I just uploaded it so it should be updated at some point today.