Re: Request for review of new redis-fdw module

From: Leon Dang <ldang(at)nahannisys(dot)com>
To: pgsql-general(at)postgresql(dot)org
Subject: Re: Request for review of new redis-fdw module
Date: 2015-01-28 08:23:29
Message-ID: 54C89C81.7090200@nahannisys.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-general

The code has been posted to https://github.com/nahanni/rw_redis_fdw

Anyone interested can test it out or review it.

cheers
Leon

Leon Dang wrote on 01/24/2015 04:17 PM:
> Hi
>
> I've implemented a completely new Redis FDW module which has little to
> do with github.com/pg-redis-fdw/redis_fdw
> <http://github.com/pg-redis-fdw/redis_fdw>; although I did take some
> inspiration from in on how the tables were to be designed but most I
> got from looking at the oracle-fdw.
>
> My redis-fdw implementation supports read and write to the Redis
> backend, so you can do insert, update, and delete. e.g. you can define
> a hash table as:
>
> table rhash (
> key text,
> field text,
> value test,
> expiry int
> ) server redserver options(tabletype 'hash');
>
> and do:
> select * from rhash where key = 'foo' and field = 'bar';
> update rhash set value = 'bahbah' where key = 'foo' and field = 'bar';
> delete from rhash where key = 'foo' and field = 'bar';
>
> I need someone experienced with postgresql's internals for FDWs to
> review my code and let me know if I've done something wrong or where I
> can optimize it a little more. Once it has been reviewed, I'll post it
> up and announce it on my github account for public consumption. But
> for now I want to make sure that the code is correct.
>
> There are some functions (Explain* and costs) which I know I haven't
> implemented properly so assistance with that is also appreciated.
>
> Please email me if you can assist with reviewing.
>
> thanks
> Leon
>

In response to

Responses

Browse pgsql-general by date

  From Date Subject
Next Message Quirin Hamp 2015-01-28 11:01:36 Re: Request for review of new redis-fdw module
Previous Message Adam Mackler 2015-01-28 04:06:30 Conflicting function name in dynamically-loaded shared library