On Tue, Jan 24, 2023 at 1:17 PM Dilip Kumar <dilipbal...@gmail.com> wrote:
>
> On Mon, Jan 23, 2023 at 6:00 PM John Naylor
> <john.nay...@enterprisedb.com> wrote:
> >
> > Attached is a rebase to fix conflicts from recent commits.
>
> I have reviewed v22-0022* patch and I have some comments.
>
> 1.
> >It also changes to the column names max_dead_tuples and num_dead_tuples
and to
> >show the progress information in bytes.
>
> I think this statement needs to be rephrased.

Could you be more specific?

> 3.
>
> We are changing the min value of 'maintenance_work_mem' to 2MB. Should
> we do the same for the 'autovacuum_work_mem'?

Yes, we should change that, too. We've discussed previously that
autovacuum_work_mem is possibly rendered unnecessary by this work, but we
agreed that that should be a separate thread. And needs additional testing
to verify.

I agree with your other comments.

--
John Naylor
EDB: http://www.enterprisedb.com

Reply via email to