Tom Lane wrote:

"Matthew T. O'Connor" <[EMAIL PROTECTED]> 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


---------------------------(end of broadcast)--------------------------- TIP 8: explain analyze is your friend

Reply via email to