From: | Dilip kumar <dilip(dot)kumar(at)huawei(dot)com> |
---|---|
To: | Abhijit Menon-Sen <ams(at)2ndQuadrant(dot)com> |
Cc: | "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>, "furuyao(at)pm(dot)nttdata(dot)co(dot)jp" <furuyao(at)pm(dot)nttdata(dot)co(dot)jp> |
Subject: | Re: pg_xlogdump --stats |
Date: | 2014-07-04 08:38:17 |
Message-ID: | 4205E661176A124FAF891E0A6BA913526634010B@szxeml509-mbs.china.huawei.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 04 July 2014 12:07, Abhijit Menon-Sen Wrote,
> -----Original Message-----
> From: Abhijit Menon-Sen [mailto:ams(at)2ndQuadrant(dot)com]
> Sent: 04 July 2014 12:07
> To: Dilip kumar
> Cc: pgsql-hackers(at)postgresql(dot)org; furuyao(at)pm(dot)nttdata(dot)co(dot)jp
> Subject: Re: [HACKERS] pg_xlogdump --stats
>
> At 2014-06-30 05:19:10 +0000, dilip(dot)kumar(at)huawei(dot)com wrote:
> >
> > Please fix these issues and send the updated patch..
> >
> > I will continue reviewing the patch..
>
> Did you get anywhere with the updated patch?
>
Patch looks fine to me, except few small comments.
1. Update this new option in "usage" function also this still have the old way { -z, --stats[=record] }
{"stats", no_argument, NULL, 'z'},
{"record-stats", no_argument, NULL, 'Z'},
2. While applying stats-newopt.dif (after applying stat2.diff), some error in merging sgml file.
patching file `doc/src/sgml/pg_xlogdump.sgml'
Hunk #1 FAILED at 181.
1 out of 1 hunk FAILED -- saving rejects to doc/src/sgml/pg_xlogdump.sgml.rej
Once you fix above erros, I think patch is ok from my side.
Thanks & Regards,
Dilip Kumar
From | Date | Subject | |
---|---|---|---|
Next Message | Abhijit Menon-Sen | 2014-07-04 08:46:42 | Re: pg_xlogdump --stats |
Previous Message | Ashutosh Bapat | 2014-07-04 07:51:30 | Re: Issue while calling new PostgreSQL command from a Java Application |