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

long type is not supported #10

Closed
solso opened this issue Jul 5, 2012 · 2 comments
Closed

long type is not supported #10

solso opened this issue Jul 5, 2012 · 2 comments

Comments

@solso
Copy link
Contributor

solso commented Jul 5, 2012

Great tool, congrats.

I'm having a problem though. While doing a

rdb -c memory mydata.rdb

On a redis 2.4.14 RDB file I get the following stacktrace:

Traceback (most recent call last):
File "/usr/local/bin/rdb", line 9, in
load_entry_point('rdbtools==0.1.2', 'console_scripts', 'rdb')()
File "/usr/local/lib/python2.6/dist-packages/rdbtools-0.1.2-py2.6.egg/rdbtools/cli/rdb.py", line 79, in main
parser.parse(dump_file)
File "/usr/local/lib/python2.6/dist-packages/rdbtools-0.1.2-py2.6.egg/rdbtools/parser.py", line 306, in parse
self.read_object(f, data_type)
File "/usr/local/lib/python2.6/dist-packages/rdbtools-0.1.2-py2.6.egg/rdbtools/parser.py", line 401, in read_object
self.read_zipmap(f)
File "/usr/local/lib/python2.6/dist-packages/rdbtools-0.1.2-py2.6.egg/rdbtools/parser.py", line 593, in read_zipmap
self._callback.hset(self._key, key, value)
File "/usr/local/lib/python2.6/dist-packages/rdbtools-0.1.2-py2.6.egg/rdbtools/memprofiler.py", line 139, in hset
if(element_length(value) > self._len_largest_element) :
File "/usr/local/lib/python2.6/dist-packages/rdbtools-0.1.2-py2.6.egg/rdbtools/memprofiler.py", line 336, in element_length
return len(element)
TypeError: object of type 'long' has no len()

solso added a commit to solso/redis-rdb-tools that referenced this issue Jul 5, 2012
sripathikrishnan added a commit that referenced this issue Jul 9, 2012
Potential fix for issue #10
@sripathikrishnan
Copy link
Owner

Fixed with above commit, closing ticket.

@solso thanks for the patch!

@solso
Copy link
Contributor Author

solso commented Jul 11, 2012

you're welcome @sripathikrishnan

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants