support redis 5.0 commands

Open dselivanov opened this issue 6 years ago • 4 comments

fix problems with json parsing for API generation
regenerate API to support redis 5.0 (redis doc commit db399bb9f3871a749bc14cc9d60761a7bfd73cf8)
add .o. .so to gitignore

Jul 17 '19 11:07 dselivanov

It seems signatures of some functions were change, thats why tests fail. I will update tests.

Jul 17 '19 12:07 dselivanov

Hi @dselivanov did you get this working? I installed your fork but I'm having some issues:

> r <- hiredis()
> r$XLEN("mystream")
Error in redis_command(ptr, cmd) : ERR unknown command 'XLEN'
> r$XADD("mystream", "*", "sensor-id", 1234)
Error in redis_command(ptr, cmd) : ERR unknown command 'XADD'

Can I help with anything?

Mar 19 '20 11:03 dseynaev

@dseynaev as I remember it was working fine. Do you have redis 5 running? looks like you redis version is < 5.0

Mar 19 '20 11:03 dselivanov

@dselivanov I was indeed blindly assuming that my local redis version was recent enough... upgrading fixed it, thanks.

Mar 19 '20 16:03 dseynaev