Add support for buffering is scripted logs

Alexey Ivanov savetherbtz at gmail.com
Mon Aug 14 18:24:39 UTC 2017


using syslog for that particular usecase seem way more elegant, customizable, and simple. As a side bonus you won't block event loop on vfs operations (open/write/close).

> On Aug 14, 2017, at 11:00 AM, Eran Kornblau <eran.kornblau at kaltura.com> wrote:
> 
>> 
>> -----Original Message-----
>> From: nginx-devel [mailto:nginx-devel-bounces at nginx.org] On Behalf Of Maxim Dounin
>> Sent: Monday, August 14, 2017 8:34 PM
>> To: nginx-devel at nginx.org
>> Subject: Re: Add support for buffering is scripted logs
>> 
>>> Ok, so is that a final 'no' for this whole feature, or is there is anything else I can do to get this feature in?
>> 
>> It is certainly not a "final no".  As I wrote in the very first comment, a) it's just a quick note, nothing more, and b) the feature is questionable.  If a good implementation will be submitted, we can consider committing it.
>> 
> That's good, I thought you were just rejecting politely :)
> 
> It would be really great if you could point me to specific parts you think look bad.
> For example, I'm guessing that you don't like the callbacks I added to open file cache,
> but I was thinking that it's better to do it this way than to duplicate large chunks of code
> and write an open file cache specific to log, please let me know if you think otherwise.
> Any feedback you can provide will be appreciated
> 
> Thanks
> 
> Eran
> 
>> --
>> Maxim Dounin
>> http://nginx.org/
>> _______________________________________________
> _______________________________________________
> nginx-devel mailing list
> nginx-devel at nginx.org
> http://mailman.nginx.org/mailman/listinfo/nginx-devel

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 842 bytes
Desc: Message signed with OpenPGP
URL: <http://mailman.nginx.org/pipermail/nginx-devel/attachments/20170814/39f96930/attachment.bin>


More information about the nginx-devel mailing list