On Mon, Jan 15, 2024 at 12:21:44PM +0000, Li, Yong wrote: > The patch looks good to me. With the context explained in the thread, > the patch is easy to understand. > The patch serves as a refactoring which pulls up common memory management > and error handling concerns into the pgarch.c. With the patch, > individual archive callbacks can focus on copying the files and leave the > boilerplate code to pgarch.c.. > > The patch applies cleanly to HEAD. “make check-world” also runs cleanly > with no error.
Thanks for reviewing. I've marked this as ready-for-committer, and I'm hoping to commit it in the near future. -- Nathan Bossart Amazon Web Services: https://aws.amazon.com
