Re: pg_autovacuum vacuum cost variables patch v2 - Mailing list pgsql-patches

From Matthew T. O'Connor
Subject Re: pg_autovacuum vacuum cost variables patch v2
Date
Msg-id 41828824.4070800@zeut.net
Whole thread Raw
In response to Re: pg_autovacuum vacuum cost variables patch v2  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-patches
Tom Lane wrote:

>"Matthew T. O'Connor" <matthew@zeut.net> writes:
>
>
>>+     if(operation == VACUUM_ANALYZE)
>>+         update_table_thresholds(dbi, tbl, VACUUM_ANALYZE);
>>+     else if(operation == VACUUM_ANALYZE)
>>+         update_table_thresholds(dbi, tbl, ANALYZE_ONLY);
>>
>>
>
>Surely that's not right ... are there any third cases here?  Why
>not just a one-liner
>    update_table_thresholds(dbi, tbl, operation);
>
>
Whoops....  I think that was both a thinko and a typeo :-)  Yes your one
line will work much better.  I'll submit another patch in a few days,
unless someone beats me to it.

Thanks for fixing that.

Matthew


pgsql-patches by date:

Previous
From: Bruce Momjian
Date:
Subject: Add buffer length setting
Next
From: Bruce Momjian
Date:
Subject: Re: Add buffer length setting