Thread: pg_autovacuum patches
Hello, I noticed that there have been a few patched submitted for pg_autovacuum that have not been applied, so I applied them locally and tested them on my Fedora box. I am resubmitting them as one single patch. Included in the attached patch: Brian Hurt's patch that fixed the truncate bug by referencing a table oid rather than it's relfilenode. Craig Boston's patch that fixes crashes on FreeBSD related to uninitialized variables. A quick patch by me to remove a log_entry call inside init_table_info() that was used for debugging by me during development and I left in by mistake. Please apply this patch to both HEAD and 7.4. Thanks much, Matthew O'Connor
ARRRGGHH.... ok, with patch this time... On Sat, 2003-11-29 at 00:17, Matthew T. O'Connor wrote: > Hello, > > I noticed that there have been a few patched submitted for pg_autovacuum > that have not been applied, so I applied them locally and tested them on > my Fedora box. I am resubmitting them as one single patch. > > Included in the attached patch: > > Brian Hurt's patch that fixed the truncate bug by referencing a table > oid rather than it's relfilenode. > > Craig Boston's patch that fixes crashes on FreeBSD related to > uninitialized variables. > > A quick patch by me to remove a log_entry call inside init_table_info() > that was used for debugging by me during development and I left in by > mistake. > > Please apply this patch to both HEAD and 7.4. > > Thanks much, > > Matthew O'Connor > > > > ---------------------------(end of broadcast)--------------------------- > TIP 5: Have you checked our extensive FAQ? > > http://www.postgresql.org/docs/faqs/FAQ.html >
Attachment
One more time... The last patch didn't have the unitialized variables stuff in it. Please disregard the previous patch and apply this one. On Sat, 2003-11-29 at 00:34, Matthew T. O'Connor wrote: > ARRRGGHH.... ok, with patch this time... > > On Sat, 2003-11-29 at 00:17, Matthew T. O'Connor wrote: > > Hello, > > > > I noticed that there have been a few patched submitted for pg_autovacuum > > that have not been applied, so I applied them locally and tested them on > > my Fedora box. I am resubmitting them as one single patch. > > > > Included in the attached patch: > > > > Brian Hurt's patch that fixed the truncate bug by referencing a table > > oid rather than it's relfilenode. > > > > Craig Boston's patch that fixes crashes on FreeBSD related to > > uninitialized variables. > > > > A quick patch by me to remove a log_entry call inside init_table_info() > > that was used for debugging by me during development and I left in by > > mistake. > > > > Please apply this patch to both HEAD and 7.4. > > > > Thanks much, > > > > Matthew O'Connor > > > > > > > > ---------------------------(end of broadcast)--------------------------- > > TIP 5: Have you checked our extensive FAQ? > > > > http://www.postgresql.org/docs/faqs/FAQ.html > > > > ______________________________________________________________________ > ---------------------------(end of broadcast)--------------------------- > TIP 2: you can get off all lists at once with the unregister command > (send "unregister YourEmailAddressHere" to majordomo@postgresql.org)
Attachment
"Matthew T. O'Connor" <matthew@zeut.net> writes:
> I noticed that there have been a few patched submitted for pg_autovacuum
> that have not been applied, so I applied them locally and tested them on
> my Fedora box. I am resubmitting them as one single patch.
Bruce seems to be vastly behind on tracking the patch queue. Not sure
why. But resubmitting existing patches in new combinations isn't gonna
make his life easier when he does start to catch up ...
regards, tom lane
Tom Lane wrote: > "Matthew T. O'Connor" <matthew@zeut.net> writes: > > I noticed that there have been a few patched submitted for pg_autovacuum > > that have not been applied, so I applied them locally and tested them on > > my Fedora box. I am resubmitting them as one single patch. > > Bruce seems to be vastly behind on tracking the patch queue. Not sure > why. But resubmitting existing patches in new combinations isn't gonna > make his life easier when he does start to catch up ... Yep. I am completing my Japan speech today. I thought I would finish earlier this week, but the xfig diagrams take more time than I thought. I am going to try to fill the queue tonight and apply it Monday. I leave for Japan on Tuesday. -- Bruce Momjian | http://candle.pha.pa.us pgman@candle.pha.pa.us | (610) 359-1001 + If your life is a hard drive, | 13 Roberts Road + Christ can be your backup. | Newtown Square, Pennsylvania 19073
Bruce Momjian <pgman@candle.pha.pa.us> writes: > I am going to try to fill the queue tonight and apply it Monday. I > leave for Japan on Tuesday. Applying a load of patches and then promptly fleeing the country doesn't sound so wise -- it sounds mighty suspicious, in fact :-) (At the same time, I can certainly testify that a 3-month long patch queue makes life more difficult for people like myself. Is there anything that others could do to reduce the patch application workload? I'd be willing to volunteer...) -Neil
Neil Conway wrote: > Bruce Momjian <pgman@candle.pha.pa.us> writes: > > I am going to try to fill the queue tonight and apply it Monday. I > > leave for Japan on Tuesday. > > Applying a load of patches and then promptly fleeing the country > doesn't sound so wise -- it sounds mighty suspicious, in fact :-) > > (At the same time, I can certainly testify that a 3-month long patch > queue makes life more difficult for people like myself. Is there > anything that others could do to reduce the patch application > workload? I'd be willing to volunteer...) Not sure. We can get others to apply them if they want, but I still have to make sure everything is getting dealt with, unless someone wants to do that too. -- Bruce Momjian | http://candle.pha.pa.us pgman@candle.pha.pa.us | (610) 359-1001 + If your life is a hard drive, | 13 Roberts Road + Christ can be your backup. | Newtown Square, Pennsylvania 19073
Bruce Momjian wrote: > Neil Conway wrote: > > Bruce Momjian <pgman@candle.pha.pa.us> writes: > > > I am going to try to fill the queue tonight and apply it Monday. I > > > leave for Japan on Tuesday. > > > > Applying a load of patches and then promptly fleeing the country > > doesn't sound so wise -- it sounds mighty suspicious, in fact :-) > > > > (At the same time, I can certainly testify that a 3-month long patch > > queue makes life more difficult for people like myself. Is there > > anything that others could do to reduce the patch application > > workload? I'd be willing to volunteer...) > > Not sure. We can get others to apply them if they want, but I still > have to make sure everything is getting dealt with, unless someone wants > to do that too. Actually, most of the delay was from saving stuff until we branched CVS, but we did that about a month ago, so the month delay was mostly that I didn't want to start throwing patch discussions into the mailing lists while we were dealing with final release issue. The last two week delay was just that I was too busy. -- Bruce Momjian | http://candle.pha.pa.us pgman@candle.pha.pa.us | (610) 359-1001 + If your life is a hard drive, | 13 Roberts Road + Christ can be your backup. | Newtown Square, Pennsylvania 19073