Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Feature/support redis server2.6 #21

Merged
merged 2 commits into from

2 participants

@typester

t/client.t fails with redis-server 2.6.x by following reasons

  • glueoutputbuf yes config is no longer supported on 2.6
  • Output format of info command changed, and AnyEvent::Redis parser wont recognize it.

This patch fixes it. I also tested this patch works with redis-server 2.4.x

@dgl dgl merged commit 4602e7d into miyagawa:master
@typester

Thanks for merging this. Could you please release new version to CPAN?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
This page is out of date. Refresh to see the latest.
Showing with 2 additions and 2 deletions.
  1. +1 −1  lib/AnyEvent/Redis.pm
  2. +1 −1  t/redis.conf.base
View
2  lib/AnyEvent/Redis.pm
@@ -290,7 +290,7 @@ sub connect {
$self->_expect($cv);
if ($command eq 'info') {
- $res = { map { split /:/, $_, 2 } split /\r\n/, $res };
+ $res = { map { split /:/, $_, 2 } grep !/^#/, split /\r\n/, $res };
} elsif ($command eq 'keys' && !ref $res) {
# Older versions of Redis (1.2) need this
$res = [split / /, $res];
View
2  t/redis.conf.base
@@ -112,5 +112,5 @@ databases 16
# Glue small output buffers together in order to send small replies in a
# single TCP packet. Uses a bit more CPU but most of the times it is a win
# in terms of number of queries per second. Use 'yes' if unsure.
-glueoutputbuf yes
+## glueoutputbuf yes
Something went wrong with that request. Please try again.