djveremix / redis

Automatically exported from code.google.com/p/redis
0 stars 0 forks source link

HINCRBY failing on 32 bit #218

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Checkout master ( ffc6b7f864dcaa58b6c5d81d7e595050fe954dec )
2. make
3. make test

What is the expected output? What do you see instead?

#179 HINCRBY against hash key created by hincrby itself                PASSED
#180 HINCRBY against hash key originally set with HSET                 PASSED
#181 HINCRBY over 32bit value                                          !!
ERROR expected
'17179869185 17179869185'
but got
'1 1'
#182 HINCRBY over 32bit value with over 32bit increment                !!
ERROR expected
'34359738368 34359738368'
but got
'0 0'
#183 HINCRBY against key with spaces (no integer encoded)              PASSED
#184 EXPIRE - don't set timeouts multiple times                        PASSED
...
#208 FLUSHDB                                                           PASSED
#209 Perform a final SAVE to leave a clean DB on disk                  PASSED

209 tests, 207 passed, 2 failed

*** WARNING!!! 2 FAILED TESTS ***

Please use labels and text to provide additional information.

$ git log -1
commit ffc6b7f864dcaa58b6c5d81d7e595050fe954dec
Author: antirez <antirez@gmail.com>
Date:   Thu Apr 1 13:13:29 2010 +0200

    Pub/Sub pattern matching capabilities

$ uname -a
Linux hierba-mala 2.6.30-02063003-generic #02063003 SMP Sat Jul 25 10:57:13
UTC 2009 i686 GNU/Linux

Original issue reported on code.google.com by anibalro...@gmail.com on 1 Apr 2010 at 3:09

GoogleCodeExporter commented 9 years ago
Confirmed.

Original comment by pcnoordh...@gmail.com on 5 Apr 2010 at 2:33

GoogleCodeExporter commented 9 years ago
This was caused by the reply not using "long long".
Fixed in: http://github.com/pietern/redis/compare/master...issue_218 

Original comment by pcnoordh...@gmail.com on 5 Apr 2010 at 2:59