On Fri, 15 Jul 2022 at 12:29, Aleksander Alekseev <aleksander@timescale.com> wrote: > Personally I didn't expect that > merging patches in this thread would take that long. They are in > "Ready for Committer" state for a long time now and there are no known > issues with them other than unit tests for SLRU [1] should be merged > first.
These patches look ready to me, including the SLRU tests.
Even though they do very little, these patches touch many aspects of the code, so it would make sense to apply these as the last step in the CF.
To encourage committers to take that next step, let's have a democratic vote on moving this forwards: +1 from me.
This set of patches no longer applies cleanly to the master branch. There are lots of hunks as well as failures. Please rebase the patches.
There are failures for multiple files including the one given below:
patching file src/backend/replication/logical/worker.c Hunk #1 succeeded at 1089 (offset 1 line). Hunk #2 succeeded at 1481 (offset 1 line). Hunk #3 succeeded at 3322 (offset 2 lines). Hunk #4 succeeded at 3493 (offset 2 lines). Hunk #5 FAILED at 4009. 1 out of 5 hunks FAILED -- saving rejects to file src/backend/replication/logical/worker.c.rej