Tom Lane wrote: > Alvaro Herrera <alvherre@xxxxxxxxxxxxxxx> writes: > > Tom Lane wrote: > >> Actually, I realized after looking at it that the new wi_tableshared > >> field is unnecessary in my patch. The only important part is that > >> knowledge of relisshared be available when we're looking for conflicting > >> workers, and that is entirely local in do_autovacuum(). I'd started the > >> patch by adding wi_tableshared, on the expectation that it would be > >> necessary, but it ain't ... > > > OK, if I understand you correctly then that is what my patch does -- the > > "sharedrel" flag in my patch is only inside do_autovacuum. Do you > > already have a test rig for this? > > I didn't attempt to replicate the problem report, if that's what you > mean; I just tested it as far as running the regression tests. OK. I'll do some more targeted testing later before pushing. -- Álvaro Herrera http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services -- Sent via pgsql-admin mailing list (pgsql-admin@xxxxxxxxxxxxxx) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-admin
