Hi, On 2018-11-28 03:06:58 +0100, Petr Jelinek wrote: > On 28/11/2018 02:14, Andres Freund wrote: > > On 2018-11-28 02:04:18 +0100, Tomas Vondra wrote: > >> Pushed and backpatched to 9.4- (same as e9edc1ba). > > > > Backpatching seems on the more aggressive end of things for an > > optimization. Could you at least announce that beforehand next time? > > > > Well, it may be optimization, but from what I've seen the problems > arising from this can easily prevent logical replication from working > altogether as reorder buffer hits OOM on bigger tables. So ISTM that it > does warrant backpatch.
I think that's a fair argument to be made. But it should be made both before the commit and in the commit message. Greetings, Andres Freund