On Fri, Oct 30, 2026 at 2:57 PM Masahiko Sawada <[email protected]> wrote: > > On Wed, Oct 28, 2026 at 3:13 PM Sami Imseih <[email protected]> wrote: > > > > > I've reviewed the patch and it mostly looks good to me. I have one > > > minor comment: > > > > > > /* Find our decoding starting point. */ > > > + set_repack_worker_setup_phase(shared, > > > + > > > PROGRESS_REPACK_PHASE_INIT_LOGICAL_DECODING); > > > DecodingContextFindStartpoint(ctx); > > > > > > While it's true that DecodingContextFindStartpoint() is the call that > > > can take a long time in this phase, I'd recommend setting the > > > INIT_LOGICAL_DECODING phase just before calling > > > CreateInitDecodingContext(). Otherwise, if CreateInitDecodingContext() > > > takes time for some reason, it would be reported as "enabling logical > > > decoding", which the documentation describes as having nothing to do > > > when effective_wal_level is already logical. > > > > Makes sense. Done in v5 > > Thank you for updating the patch! It looks good to me, so I'll push it > early next week, barring objections.
Sorry for the late notice. I pushed this patch on Monday (51ae21aa31). Regards, -- Masahiko Sawada Amazon Web Services: https://aws.amazon.com
