Announcing a new PHP client for sentinel

[james cauwelier]

Hi All,

I've been busy writing a sentinel client in PHP that is able to support different redis clients through use of the adapter pattern.  There is still a lot to be done, but I feel like enough has been done to start experimenting it and using it since the master discovery has been implemented.  There are plenty of unit tests and a decent amount of integration tests as well. 

Although I need to do some cleanup of variable naming, add documentation on how to use the library and how to run the integration tests I feel that people can use the integration tests themselves as documentation for now.

My short-term goals for the library are listed at https://github.com/Sparkcentral/php-redis-sentinel/issues/milestones?with_issues=no

Whoever is interested is invited to fork and experiment with this.  All feedback is appreciated.

Thanks!

[james cauwelier]

In case it isn't obvious, here is the link to the library itself instead of the milestones:

https://github.com/Sparkcentral/php-redis-sentinel

[Geoffrey Hoffman]

Hey James, 

Awesome work. I probably should preface this by saying (sadly) I don't have immediate need to use your library, it's just killer code and I thank you greatly for making it open source.

Could you help me understand `namespace Redis`?

In the past I've installed the PHPRedis extension (native C implementation) -- it creates `Redis` functions that appear native to PHP implementation. 

You're using Predis I see? I am kind of surprised, in that, I would've thought most PHP Redis users would opt for the C PHP Extension for speed, which makes me wonder, or at least for me to even try your code, I would need it to be in a different namespace, since I have PHPRedis which creates Redis:: already… right?

The code you present is so clean and well thought out that it makes me wonder if I am misunderstanding namespaces in PHP 5.4+ altogether.

--
You received this message because you are subscribed to the Google Groups "Redis DB" group.
To unsubscribe from this group and stop receiving emails from it, send an email to redis-db+u...@googlegroups.com.
To post to this group, send email to redi...@googlegroups.com.
Visit this group at http://groups.google.com/group/redis-db.
For more options, visit https://groups.google.com/d/optout.

[James Cauwelier]

Hi Geoffrey,

Thank you for the kind words.

And you're right about the Redis namespace.  It should not collide with existing namespaces.  I'll make sure I fix that while I am still labeling it as alpha software.

For a painter to create something truly beautiful, he sometimes needs to step away from the canvas.  Thank you for doing that in my place.

Sent from my iPhone

You received this message because you are subscribed to a topic in the Google Groups "Redis DB" group.
To unsubscribe from this topic, visit https://groups.google.com/d/topic/redis-db/YKnl3il8RBc/unsubscribe.
To unsubscribe from this group and all its topics, send an email to redis-db+u...@googlegroups.com.