On Thu, 2007-10-25 at 13:41 -0300, Alvaro Herrera wrote: > Simon Riggs wrote: > > On Thu, 2007-10-25 at 14:45 +0000, Alvaro Herrera wrote: > > > Log Message: > > > ----------- > > > Extract catalog info for error reporting before an error actually happens. > > > Also, remove redundant reset of for-wraparound PGPROC flag. > > > > Just noticed you've made these changes. I was working on them, but > > hadn't fully tested the patch because of all the different touch points. > > Sorry for the delay. > > > > Would you like me to refresh my earlier patch against the newly > > committed state (or did you commit that aspect already as well?). > > I am doing just that. I'll post it soon.
OK thanks. > FWIW I disagree with cancelling just any autovac work automatically; in > my patch I'm only cancelling if it's analyze, on the grounds that if > you have really bad luck you can potentially lose a lot of work that > vacuum did. We can relax this restriction when we have cancellable > vacuum. That was requested by others, not myself, but I did agree with the conclusions. The other bad luck might be that you don't complete some critical piece of work in the available time window because an automated job kicked in. -- Simon Riggs 2ndQuadrant http://www.2ndQuadrant.com ---------------------------(end of broadcast)--------------------------- TIP 9: In versions below 8.0, the planner will ignore your desire to choose an index scan if your joining column's datatypes do not match
