Varnishtop on the new logging API

[Martin Blix Grydeland]

Hi,

This patch has now been merged. Thanks for the contribution!

If you'd like to have a look at varnishhist too that would be much
appreciated. Have a look at the commit history for some pointers as to
missing items in the original patch.

Regards,
Martin Blix Grydeland


On 13 January 2014 09:55, Martin Blix Grydeland <martin at varnish-software.com
> wrote:

> Hi Guillaume,
>
> I'll come back to you with a proper review and reply to this some time
> next week. This week I am away on a business trip all week and connectivity
> will be mostly offline.
>
> Martin
> 9. jan. 2014 16:44 skrev "Guillaume Quintard" <
> guillaume.quintard at smartjog.com> følgende:
>
>  Ah, I was wondering about it too. Martin, feel free to give me pointer
>> about what you want changed, I see a few spots that could be more elegant
>> or consistent, but you'll have probably more to say.
>>
>> On 01/09/2014 05:41 PM, Martin Blix Grydeland wrote:
>>
>> Not looked at it in detail yet, though first impression looks good. Will
>> be going in with some tweaks. Thanks for the reminder though :-)
>>
>> Main
>> 9. jan. 2014 17:23 skrev "Poul-Henning Kamp" <phk at phk.freebsd.dk>
>> følgende:
>>
>>>
>>> >As said on IRC, here's a shot at porting varnishtop on the new logging
>>> >API. I didn't change much, maybe not enough (RB tree freeing?), but it
>>> >seems to work.
>>>
>>> I'm wading through my inbox, did Martin look at this patch ?
>>>
>>> --
>>> Poul-Henning Kamp       | UNIX since Zilog Zeus 3.20
>>> phk at FreeBSD.ORG         | TCP/IP since RFC 956
>>> FreeBSD committer       | BSD since 4.3-tahoe
>>> Never attribute to malice what can adequately be explained by
>>> incompetence.
>>>
>>> _______________________________________________
>>> varnish-dev mailing list
>>> varnish-dev at varnish-cache.org
>>> https://www.varnish-cache.org/lists/mailman/listinfo/varnish-dev
>>>
>>
>>
>> --
>> Guillaume Quintard
>>
>>


-- 
<http://varnish-software.com>*Martin Blix Grydeland*
Senior Developer | Varnish Software AS
Cell: +47 21 98 92 60
We Make Websites Fly!
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://www.varnish-cache.org/lists/pipermail/varnish-dev/attachments/20140129/c6509584/attachment.html>