Re: [PATCHES] HOT documentation README

2007-09-13 Thread Pavan Deolasee
On 9/12/07, Pavan Deolasee <[EMAIL PROTECTED]> wrote: > > > One change that is worh mentioning > and discussing is that we don't follow HOT chains while fetching tuples > during > autoanalyze and autoanalyze would consider all such tuples as DEAD. > In the worst case when all the tuples in the tabl

Re: [PATCHES] HOT documentation README

2007-09-12 Thread Pavan Deolasee
On 9/12/07, Tom Lane <[EMAIL PROTECTED]> wrote: > > > > This seems all wrong to me. We'd only be considering freezing a tuple > whose xmin precedes the global xmin. If it has a predecessor, that > predecessor has xmax equal to this one's xmin, therefore also preceding > global xmin, therefore it

Re: [PATCHES] HOT documentation README

2007-09-12 Thread Tom Lane
"Pavan Deolasee" <[EMAIL PROTECTED]> writes: > On 9/12/07, Tom Lane <[EMAIL PROTECTED]> wrote: >> XXX doesn't the above completely break the anti-wraparound guarantees? >> And why couldn't we avoid the problem by pruning the previous tuple, >> which is surely dead? > We preserve anti-wraparound gu

Re: [PATCHES] HOT documentation README

2007-09-12 Thread Tom Lane
"Pavan Deolasee" <[EMAIL PROTECTED]> writes: > On 9/12/07, Tom Lane <[EMAIL PROTECTED]> wrote: >> XXX the above being the case, why should we bother with all this extra >> mechanism? Seems we could dispense with "not ready for inserts". > Without this additional step, while we are building the in

Re: [PATCHES] HOT documentation README

2007-09-12 Thread Pavan Deolasee
On 9/12/07, Tom Lane <[EMAIL PROTECTED]> wrote: > > > VACUUM FULL > --- > > To make vacuum full work, any DEAD tuples in the middle of an update > chain need to be removed (see comments at the top of > heap_prune_hotchain_hard() for details). Vacuum full performs a more > aggressive pruning

Re: [PATCHES] HOT documentation README

2007-09-12 Thread Pavan Deolasee
On 9/12/07, Tom Lane <[EMAIL PROTECTED]> wrote: > > > > VACUUM > -- > > There is little change to regular vacuum. It removes dead HOT tuples, > like pruning does, and cleans up any redirected dead line pointers. > > In lazy vacuum, we must not freeze a tuple that is in the middle of an > update

Re: [PATCHES] HOT documentation README

2007-09-11 Thread Pavan Deolasee
On 9/12/07, Tom Lane <[EMAIL PROTECTED]> wrote: > > Bruce Momjian <[EMAIL PROTECTED]> writes: > > Current README.HOT version: > > ftp://momjian.us/pub/postgresql/mypatches/README.HOT > > I've done some further editorializing and wordsmithing on this, > and marked with XXX some areas that seem

Re: [PATCHES] HOT documentation README

2007-09-11 Thread Tom Lane
Bruce Momjian <[EMAIL PROTECTED]> writes: > Current README.HOT version: > ftp://momjian.us/pub/postgresql/mypatches/README.HOT I've done some further editorializing and wordsmithing on this, and marked with XXX some areas that seem like they're still red flags (or at least inadequately descr

Re: [PATCHES] HOT documentation README

2007-09-04 Thread Gregory Stark
"Pavan Deolasee" <[EMAIL PROTECTED]> writes: > It would be worth mentioning that columns appearing in predicates > of partial indexes and expressions of expression indexes are also > checked. If any of these columns are changed, HOT update is not done. In discussion a question arose. I don't thi

Re: [PATCHES] HOT documentation README

2007-09-04 Thread Bruce Momjian
Pavan Deolasee wrote: > It would be worth mentioning that columns appearing in predicates > of partial indexes and expressions of expression indexes are also > checked. If any of these columns are changed, HOT update is not done. Added now: Tuples are checked against expression and

Re: [PATCHES] HOT documentation README

2007-09-04 Thread Bruce Momjian
Gregory Stark wrote: > > "Bruce Momjian" <[EMAIL PROTECTED]> writes: > > > I have taken this, and Pavan's documentation about CREATE INDEX, and > > worked up an updated README. Comments? Corrections? > > Oh, you I think the CREATE INDEX documentation you refer to was actually the > one I sugge

Re: [PATCHES] HOT documentation README

2007-09-04 Thread Pavan Deolasee
On 9/4/07, Bruce Momjian <[EMAIL PROTECTED]> wrote: > > > I have taken this, and Pavan's documentation about CREATE INDEX, and > worked up an updated README. Comments? Corrections? Thanks Bruce, Heikki, Greg for helping me with the documentation. > The requirements for doing a HOT update is

Re: [PATCHES] HOT documentation README

2007-09-04 Thread Pavan Deolasee
On 9/4/07, Gregory Stark <[EMAIL PROTECTED]> wrote: > > > NOTE: This is something likely to change. Current discussions are leading > towards handling DELETE_IN_PROGRESS and INSERT_IN_PROGRESS from other > transactions. We would do this by waiting until the transactions owning > those > tuples exi

Re: [PATCHES] HOT documentation README

2007-09-04 Thread Gregory Stark
"Bruce Momjian" <[EMAIL PROTECTED]> writes: > I have taken this, and Pavan's documentation about CREATE INDEX, and > worked up an updated README. Comments? Corrections? Oh, you I think the CREATE INDEX documentation you refer to was actually the one I suggested. A few tweaks: > (If we find a

Re: [PATCHES] HOT documentation README

2007-09-04 Thread Heikki Linnakangas
Bruce Momjian wrote: > Heikki Linnakangas wrote: >> Tom Lane wrote: >>> "Pavan Deolasee" <[EMAIL PROTECTED]> writes: Please see the version 14 of HOT patch attached. >>> I expected to find either a large new README, or some pretty substantial >>> additions to existing README files, to document

Re: [PATCHES] HOT documentation README

2007-09-04 Thread Gregory Stark
"Bruce Momjian" <[EMAIL PROTECTED]> writes: > Heikki Linnakangas wrote: > >> Here's an updated version of the README I posted earlier. It now >> reflects the changes to how pruning works. > > I have taken this, and Pavan's documentation about CREATE INDEX, and > worked up an updated README. Comme

[PATCHES] HOT documentation README

2007-09-03 Thread Bruce Momjian
Heikki Linnakangas wrote: > Tom Lane wrote: > > "Pavan Deolasee" <[EMAIL PROTECTED]> writes: > >> Please see the version 14 of HOT patch attached. > > > > I expected to find either a large new README, or some pretty substantial > > additions to existing README files, to document how this all works