@Brar note the test failures when running the replication tests against PG18 beta1... I suspect it might just be test issues (where we assume a very specific message order which changed), but who knows...

I propose we merge this for now without adding PG18 as a mandatory passing CI config, and get it green afterwards...

Most of the errors are related to transaction streaming.
The assertions fail because transaction streaming doesn't happen although it should.
It's probably related to logical_decoding_work_mem configuration or something around this.

It's probably related to logical_decoding_work_mem configuration or something around this.

I just saw that it's only happening in the LogicalDecodingMessage() test, so streaming generally seems to work but not for logical decoding messages.
That's surprising.
Maybe a bug in the PG beta?
Unfortunately I don't have much time to investigate this at the moment.

Sure thing - I opened #6158 to track this. I don't think it's super urgent, though especially if there may be a problem on the PG18 we'd want to report that.

@vonzshik @NinoFloris i propose just merging this PR as-is, with the failing PG18 config. It doesn't block us from merging PRs (I haven't added PG18 as a mandatory check), and this way we'll get a constant reminder too ;)