ITAGAKI Takahiro wrote:
> 
> Alvaro Herrera <[EMAIL PROTECTED]> wrote:
> 
> > 1. changing SIGINT so that it cancels the current table instead of
> > shutting down the entire worker.

I applied this part of the patch, thanks.

In passing I noticed that apparently we are leaking memory, because the
vacuum memory context is created with a parent of PortalContext, which
ISTM to be NULL in autovacuum.  So when we cancel the vacuuming work, we
never delete or reset that context.

I think what we should be doing is creating a context to act as
PortalContext, and reset it after each vacuuming operation.

-- 
Alvaro Herrera                  http://www.amazon.com/gp/registry/5ZYLFMCVHXC
"El que vive para el futuro es un iluso, y el que vive para el pasado,
un imb├ęcil" (Luis Adler, "Los tripulantes de la noche")

---------------------------(end of broadcast)---------------------------
TIP 6: explain analyze is your friend

Reply via email to