Andres Freund <andres@2ndquadrant.com> writes:
> Solution I)
> Change ReorderBufferCommit() to not call the begin()/commit() callbacks
> if no change() callback has been called. Technically that's trivial, but
> I don't think that'd end up being a better behaviour.
> Solution II)
> Disable autovacuum/analyze for the tables in the regression tests. We
> test vacuum explicitly, so we wouldn't loose too much test coverage.
> Solution III)
> Mark transactions that happen purely internally as such, using a
> xl_xact_commit->xinfo flag. Not particularly pretty and the most
> invasive of the solutions.
> I'm favoring II) so far... Other opinions?
You mean disable autovac only in the contrib/test_decoding check run,
right? That's probably OK since it's tested in the main regression
runs.
regards, tom lane