Johannes Schlüter wrote:
> On Thu, 2010-03-18 at 22:10 -0600, Raphael Geissert wrote:
>> At the moment dba is the best place to add support for tokyo cabinets
>> without introducing major features (or a pecl extension).
> 
> "without introducing major features" doesn't sound like a good way for
> architecture decisions. This sounds a bit like "let's do it the way we
> can trick the process" to me.

Well, it is just another handler :)

> And there is a pecl extension: http://pecl.php.net/package/tokyo_tyrant
> 

Which doesn't address the same problem. Tokyo tyrant is a DBMS internally 
using tokyo cabinets. There are cases where running tokyo tyrant is either 
not possible, an overkill or not the right solution to a problem, hence the 
request of the dba handler.

>> That'd be great. In fact, the interface isn't really bad. It is surely
>> missing multiple features, some of which are only supported by a couple
>> of dbms, but the code behind is what really needs to be improved.
> 
> Thanks for volunteering! :-)
> 

Meh :)

>> > In addition, I wrote a patch for Tokyo Cabinet support
>> > for ext/dba a while back, which Scott applied to PHP6 - I was rather
>> > hoping it'd end up in 5.3.x at some point, but he's been too busy. I'll
>> > dig out the patch if anyone here wants it, I haven't karma to apply it.
>> 
>> Cool.
>> 
>> Johannes, would it be ok to merge support for tokyo cabinets into
>> PHP_5_3?
> 
> I didn't review it, yet, it sounds like a bit much, putting the review
> on my todo.

Would be great if it is merged. Thanks in advance.

Cheers,
-- 
Raphael Geissert

-- 
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to