Re: [Patch V2 1/1] [MINOR] Enhancement to stats page to provide information of last session time.

2014-02-07 Thread Willy Tarreau
Hi Bhaskar, On Wed, Feb 05, 2014 at 12:10:09PM -0500, Bhaskar Maddala wrote: Hello, Resubmitting the patch with changed to address concerns from previous attempt Updates from previous submission (a) using unsigned long for last session time stamp instead of timeval struct (b)

Re: [Patch V2 1/1] [MINOR] Enhancement to stats page to provide information of last session time.

2014-02-07 Thread Bhaskar Maddala
Sorry that was my mistake. I didn't not check how some of the other stats were done. Please make the change. Thank you On Feb 7, 2014 6:52 PM, Willy Tarreau w...@1wt.eu wrote: Hi Bhaskar, On Wed, Feb 05, 2014 at 12:10:09PM -0500, Bhaskar Maddala wrote: Hello, Resubmitting the patch

Re: [Patch V2 1/1] [MINOR] Enhancement to stats page to provide information of last session time.

2014-02-07 Thread Willy Tarreau
On Fri, Feb 07, 2014 at 07:07:46PM -0500, Bhaskar Maddala wrote: Sorry that was my mistake. I didn't not check how some of the other stats were done. Please make the change. Great. Done, pushed and deployed : http://demo.1wt.eu/ I had to add the empty field also on the listener stats which

[Patch V2 1/1] [MINOR] Enhancement to stats page to provide information of last session time.

2014-02-05 Thread Bhaskar Maddala
Hello, Resubmitting the patch with changed to address concerns from previous attempt Updates from previous submission (a) using unsigned long for last session time stamp instead of timeval struct (b) avoiding tv_now and using now Thanks Bhaskar 0001_last_session_date_stats.patch