Re: Leftover reference to replacement selection 1 run case

2017-12-12 Thread Robert Haas
On Tue, Dec 12, 2017 at 6:44 PM, Peter Geoghegan wrote: > Yes, you have that right. Per dumptuples(), even the zero tuple run edge > case will still write a run marker, and will therefore still consume a tape. > We must have at least two initial runs to merge. (though dummy runs for non > final me

Re: Leftover reference to replacement selection 1 run case

2017-12-12 Thread Peter Geoghegan
Yes, you have that right. Per dumptuples(), even the zero tuple run edge case will still write a run marker, and will therefore still consume a tape. We must have at least two initial runs to merge. (though dummy runs for non final merges are a slightly different matter.) -- Peter Geoghegan (Sent

Re: Leftover reference to replacement selection 1 run case

2017-12-12 Thread Robert Haas
On Fri, Dec 8, 2017 at 6:41 PM, Peter Geoghegan wrote: > While reviewing the parallel CREATE INDEX patch, I noticed that commit > 8b304b8b omitted to remove a comment that it made obsolete. > > Attached patch removes the comment. I had to think about this for a while. I could see that the commen

Leftover reference to replacement selection 1 run case

2017-12-08 Thread Peter Geoghegan
While reviewing the parallel CREATE INDEX patch, I noticed that commit 8b304b8b omitted to remove a comment that it made obsolete. Attached patch removes the comment. -- Peter Geoghegan From a52ca388cb21f797bcfe3a1b15cc9168e5d26f41 Mon Sep 17 00:00:00 2001 From: Peter Geoghegan Date: Fri, 8 Dec