Re: Decoding of (nearly) empty transactions and regression tests

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