Love Fuel?    Donate

FuelPHP Forums

Ask your question about FuelPHP in the appropriate forum, or help others by answering their questions.
About the redis class in the future 1.3 release
  • Hello I tried the new Redis class from the 1.3 dev branch.
    The pipeline function works, but seems useless -unless I haven't understand it correctly. The comments say the execute() function opens the connection to Redis and then executes the pipelined commands. But it's not true; the connection is opened only once, in the constructor. The execute() method just writes the commands in the socket. So there is no performance gain using the pipeline. In fact there is a minor performance degradation: when invoking a unique Redis command, the execute() method is called and does a useless iteration. So, from my point of view, this new version of the Redis class is less good than the previous one. Let me know if my understanding is correct. If you have any idea, I would be glad to help improving this feature.
  • We're always looking for improvements, so if you have them, by all means. Fork the repo, makes your changes, and send a pull request.
  • I surely will, but I would have been interested in discussing with the author of this class before doing it.
  • We haven't written it, it's a copy of https://github.com/jdp/redisent, adapted to work with FuelPHP. I did the latest merge, so it could be that I've misinterpreted the new way of working, I don't use redis myself.

Howdy, Stranger!

It looks like you're new here. If you want to get involved, click one of these buttons!

In this Discussion