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

Already on GitHub? Sign in to your account

Change redis-check-dump to exit(1) on errors #747

wants to merge 1 commit into


None yet
1 participant

nuxi commented Nov 2, 2012

This allows the return code of redis-check-dump to be used in scripts.

nuxi commented Nov 2, 2012

I admit the if(num_errors == 0 ) { exit(1) } feels dirty, but I didn't think CRCs counted as unprocessed opcodes either.

Alternatively you could ERROR() print the CRC message which would skip the later opcode code printout. Or just increment num_errors;

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment