On Mon, Sep 7, 2026 at 9:04 PM vignesh C <[email protected]> wrote: > > On Thu, 3 Sept 2026 at 16:15, Mihail Nikalayeu > <[email protected]> wrote: > > > > Zhijie, Amit, thanks for the reviews! > > > > > we shall mention in the comments atop the old function that it should > > > not be used in new code anymore > > > > Done. > > > > Couple of minor comments: > 1) I was able to compile without this header inclusion: > --- a/src/backend/replication/logical/worker.c > +++ b/src/backend/replication/logical/worker.c > @@ -249,6 +249,7 @@ > > #include "access/genam.h" > #include "access/commit_ts.h" > +#include "access/htup_details.h" > #include "access/table.h" >
Fixed in the attached. Apart from this I changed multiple comments to make those clear. One notable change is, I moved the newly added boolean after localindexoid as it reads better there because then we don't need to forward reference the fields. For back-branches, if it needs to be moved to an earlier location then we can do that in those versions but for HEAD and 19, the new location seems better. Also, shall we keep just one test, say Drop Index Concurrently instead of two as both tests do the same thing in a slightly different way? I have not done that but if you agree please update the patch accordingly. -- With Regards, Amit Kapila.
v4-0001-Fix-tuple-search-during-apply-after-concurrent-in.patch
Description: Binary data
