Re: [PATCH] store spool directory support

Hi,

with this patch the way is then open for implementing store subsystem in a  
database.

Am 26.10.2006, 13:49 Uhr, schrieb Enver ALTIN <[email protected]>:

> Hi Alex,
>
> On Tue, 2006-10-17 at 18:20 +0200, Alexander Malysh wrote:
>> please find at https://olek.homeip.net/~alex/store-spool.patch patch  
>> that
>> adds additinaly support for store spool directory.
>> I added two new config options:
>> 'store-type' - allow us to define which type to use (file, spool)
>> 'store-location' - allow setting either file or spool location
>>
>> Option 'store-file' is deprecated but still supported with a warning  
>> that
>> two above options should be used.
>> By default if no 'store-type' defined store file is used.
>>
>> Advantages of the store spool is:
>> 1) each message stored as one file, so it's easier to drop one specific
>> message from the queue
>> 2) no need to have queued messages twice in the memory
>> 3) no locking required -> no race conditions as is the case for store  
>> file
>> 4) some other that I don't know yet :)
>>
>> Please test it and give me feedback.
>
> Sorry for the delay. I have briefly read over the patch, and it looks
> good to me (that means I'm +1).
>
> I wonder why don't we just use a database. Hrm. Maybe I'll give it a try
> sometime.



-- 
Thanks,
Alex