On 19 November 2013 22:19, Sawada Masahiko Wrote

> >>
> >> Thank you for comment.
> >> Actually, I had thought to add separate parameter.
> >
> > I think that he said that if you can proof that amount of WAL is
> > almost same and without less performance same as before, you might
> not
> > need to separate parameter in your patch.
> >
> 
> Thanks!
> I took it wrong.
> I think that there are quite a few difference amount of WAL.
> 
> > Did you test about amount of WAL size in your patch?
> 
> Not yet. I will do that.

1. Patch applies cleanly to master HEAD.
2. No Compilation Warning.
3. It works as per the patch expectation.

Some Suggestion:
1. Add new WAL level ("all") in comment in postgresql.conf 
   wal_level = hot_standby                         # minimal, archive, or 
hot_standby


Performance Test Result:
    Run with pgbench for 300 seconds

    WAL level :         hot_standby
    WAL Size  :   111BF8A8
    TPS       :   125

    WAL level :         all
    WAL Size  :   11DB5AF8
    TPS       :   122 

    * TPS is almost constant but WAL size is increased around 11M.

This is the first level of observation, I will continue to test few more 
scenarios including performance test on standby.

Regards,
Dilip Kumar



                






-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to