On Tue, May 12, 2020 at 10:26 PM Amit Kapila <amit.kapil...@gmail.com> wrote: > I was trying to say that tablespace listing will happen under > PROGRESS_BASEBACKUP_PHASE_WAIT_CHECKPOINT phase which could be a > problem if it is a costly operation but as you said it is pretty cheap > so I think we don't need to bother about that. > > Apart from the above point which I think we don't need to bother, both > your patches look good to me.
OK, good. Let's see if anyone else feels differently about this issue or wants to raise anything else. If not, I'll plan to commit these patches after we branch. Thanks for the review. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company
