Check out jedis, see whether we can replace jredis with it

Bug #837908 reported by Muharem Hrnjadovic on 2011-08-31
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenQuake (deprecated)
High
Unassigned

Bug Description

Barring a decision to switch to MongoDB and drop redis altogether, we should check out jedis [1], the maintained and recommended java bindings for redis and see whether we can replace jredis.

[1] https://github.com/xetorthio/jedis

Changed in openquake:
status: New → Confirmed
importance: Undecided → High
tags: added: 12.04-lts nosql
Muharem Hrnjadovic (al-maisan) wrote :

Hmm .. from https://github.com/xetorthio/jedis :

"Jedis is fully compatible with redis 2.0.0." .. sent the author a message and asked about redis-2.2 compatibility ..

On Wed, Aug 31, 2011 at 08:58:06AM -0000, Muharem Hrnjadovic wrote:
> Hmm .. from https://github.com/xetorthio/jedis :
>
> "Jedis is fully compatible with redis 2.0.0." .. sent the author a
> message and asked about redis-2.2 compatibility ..

According to [1] redis 2.2 is a drop in replacement for 2.0, so there
should be no need for a specific client for 2.2.

But it's better to be sure :)

[1] https://raw.github.com/antirez/redis/2.2/00-RELEASENOTES

Not sure how well jedis is crafted but I remember us having failures when we used jredis in conjunction with redis-server revisions newer then 2.0.X

Besides, the 12.04 LTS release we are preparing for is very likely to ship redis-sever rev. 2.4

Muharem Hrnjadovic (al-maisan) wrote :

Response from the author (https://github.com/xetorthio):

{{
Re: Does jedis work with redis-server 2.2?

Yes it should. There might be some command missing, but other than that,
Jedis is 100% compatible.
}}

summary: - Check ot jedis, see whether we can replace jredis with it
+ Check out jedis, see whether we can replace jredis with it
Muharem Hrnjadovic (al-maisan) wrote :

This is not needed any more, jredis is doing the trick for the time being.

Changed in openquake:
status: Confirmed → Invalid
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers