|From:||Abhijit Menon-Sen <ams(at)2ndQuadrant(dot)com>|
|To:||Dilip kumar <dilip(dot)kumar(at)huawei(dot)com>|
|Subject:||Re: pg_xlogdump --stats|
|Views:||Raw Message | Whole Thread | Download mbox | Resend email|
At 2014-06-30 05:19:10 +0000, dilip(dot)kumar(at)huawei(dot)com wrote:
> I have started reviewing the patch..
> 1. Patch applied to git head cleanly.
> 2. Compiled in Linux -- Some warnings same as mentioned by furuyao
I've attached an updated version of the patch which should fix the
warnings by using %zu.
> 3. Some compilation error in windows
> .\contrib\pg_xlogdump\pg_xlogdump.c(1002) : error C2065: 'optional_argument' : undeclared identifier
> .\contrib\pg_xlogdump\pg_xlogdump.c(1002) : error C2099: initializer is not a constant
> optional_argument should be added to getopt_long.h file for windows.
Hmm. I have no idea what to do about this. I did notice when I wrote the
code that nothing else used optional_argument, but I didn't realise that
it wouldn't work on Windows.
It may be that the best thing to do would be to avoid using
optional_argument altogether, and have separate --stats and
--stats-per-record options. Thoughts?
> Please fix these issues and send the updated patch..
I've also attached a separate patch to factor out the identify_record
function into rm_identify functions in the individual rmgr files, so
that the code can live next to the respective _desc functions.
This allows us to remove a number of #includes from pg_xlogdump, and
makes the code a bit more straightforward to read.
|Next Message||Abhijit Menon-Sen||2014-06-30 06:49:26||Re: pg_xlogdump --stats|
|Previous Message||Rajeev rastogi||2014-06-30 06:17:13||Re: psql: show only failed queries|