On Wed, Sep 30, 2020 at 1:12 PM Dilip Kumar <dilipbal...@gmail.com> wrote: > > On Fri, Sep 25, 2020 at 4:33 PM Amit Kapila <amit.kapil...@gmail.com> wrote: > > > > On Thu, Sep 24, 2020 at 5:44 PM Amit Kapila <amit.kapil...@gmail.com> wrote: > > > > I have done some more testing of this patch especially for the case > > where we spill before streaming the transaction and found everything > > is working as expected. Additionally, I have changed a few more > > comments and ran pgindent. I am still not very sure whether we want to > > display physical slots in this view as all the stats are for logical > > slots but anyway we can add stats w.r.t physical slots in the future. > > I am fine either way (don't show physical slots in this view or show > > them but keep stats as 0). Let me know if you have any thoughts on > > these points, other than that I am happy with the current state of the > > patch. > > IMHO, It will make more sense to only show the logical replication > slots in this view. >
Okay, Sawada-San, others, do you have any opinion on this matter? -- With Regards, Amit Kapila.