Re: Calculate total_table_pages after set_base_rel_sizes()
Tom Lane <tgl@sss.pgh.pa.us>
From: Tom Lane <tgl@sss.pgh.pa.us>
To: David Rowley <david.rowley@2ndquadrant.com>
Cc: Edmund Horner <ejrh00@gmail.com>,
PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2018-11-07T17:17:57Z
Lists: pgsql-hackers
David Rowley <david.rowley@2ndquadrant.com> writes: > [ v2-0001-Calculate-total_table_pages-after-set_base_rel_si.patch ] Pushed with cosmetic adjustments. The reason it's okay to not check for appendrels in this loop is now quite different from the reason it was okay before, so I didn't like the fact that you'd just cut-and-pasted the same comment. Also, you did s/brel/rel/ in the transferred code, which I would not have objected to except you were inserting it into a function that (a) already had a local variable "rel", causing a shadowing situation, and (b) had an identical loop just above this that used the "brel" notation. So I changed that back. regards, tom lane
Commits
-
Postpone calculating total_table_pages until after pruning/exclusion.
- c6e4133fae1f 12.0 landed