Re: Lock compatibility matrix

Lists: pgsql-patches
From: "Pavan Deolasee" <pavan(dot)deolasee(at)gmail(dot)com>
To: pgsql-patches(at)postgresql(dot)org
Subject: Lock compatibility matrix
Date: 2007-01-30 09:52:32
Message-ID: 2e78013d0701300152s25f4727fh23d5a27a9b9ef985@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-patches

I had this in a different form, but reworked so that it matches the doc
patch that Teodor submitted earlier. I think it would be good to have this
information in the lock.h file as well.

Thanks,
Pavan

--

EnterpriseDB http://www.enterprisedb.com

Attachment Content-Type Size
lock-compatibility.patch application/octet-stream 2.0 KB

From: Peter Eisentraut <peter_e(at)gmx(dot)net>
To: pgsql-patches(at)postgresql(dot)org
Cc: "Pavan Deolasee" <pavan(dot)deolasee(at)gmail(dot)com>
Subject: Re: Lock compatibility matrix
Date: 2007-01-30 10:09:01
Message-ID: 200701301109.02982.peter_e@gmx.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-patches

Pavan Deolasee wrote:
> I had this in a different form, but reworked so that it matches the
> doc patch that Teodor submitted earlier. I think it would be good to
> have this information in the lock.h file as well.

Why would we want to have two redundant copies of the same information?

--
Peter Eisentraut
http://developer.postgresql.org/~petere/


From: "Pavan Deolasee" <pavan(dot)deolasee(at)gmail(dot)com>
To: "Peter Eisentraut" <peter_e(at)gmx(dot)net>
Cc: pgsql-patches(at)postgresql(dot)org
Subject: Re: Lock compatibility matrix
Date: 2007-01-30 11:06:57
Message-ID: 2e78013d0701300306va013260xc423d0ad6bc5dbdc@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-patches

On 1/30/07, Peter Eisentraut <peter_e(at)gmx(dot)net> wrote:
>
> Pavan Deolasee wrote:
> > I had this in a different form, but reworked so that it matches the
> > doc patch that Teodor submitted earlier. I think it would be good to
> > have this information in the lock.h file as well.
>
> Why would we want to have two redundant copies of the same information?

IMHO its useful to have this information in the source code, just like many
other comments. It improves the readability of the code while documentation
acts as a reference.

But I am not sure whats the generally accepted practice for PostgresQL,
so I may be wrong here.

Thanks,
Pavan

--

EnterpriseDB http://www.enterprisedb.com


From: "Simon Riggs" <simon(at)2ndquadrant(dot)com>
To: "Peter Eisentraut" <peter_e(at)gmx(dot)net>
Cc: <pgsql-patches(at)postgresql(dot)org>, "Pavan Deolasee" <pavan(dot)deolasee(at)gmail(dot)com>
Subject: Re: Lock compatibility matrix
Date: 2007-01-30 20:17:15
Message-ID: 1170188236.3681.226.camel@silverbirch.site
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-patches

On Tue, 2007-01-30 at 11:09 +0100, Peter Eisentraut wrote:
> Pavan Deolasee wrote:
> > I had this in a different form, but reworked so that it matches the
> > doc patch that Teodor submitted earlier. I think it would be good to
> > have this information in the lock.h file as well.
>
> Why would we want to have two redundant copies of the same information?

The lock information is not available anywhere in the form of a matrix.

I've personally found a matrix useful for application design, though
that hasn't influenced Pavan's independent creation of exactly that.

--
Simon Riggs
EnterpriseDB http://www.enterprisedb.com


From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: "Simon Riggs" <simon(at)2ndquadrant(dot)com>
Cc: "Peter Eisentraut" <peter_e(at)gmx(dot)net>, pgsql-patches(at)postgresql(dot)org, "Pavan Deolasee" <pavan(dot)deolasee(at)gmail(dot)com>
Subject: Re: Lock compatibility matrix
Date: 2007-01-30 20:31:37
Message-ID: 9240.1170189097@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-patches

"Simon Riggs" <simon(at)2ndquadrant(dot)com> writes:
> On Tue, 2007-01-30 at 11:09 +0100, Peter Eisentraut wrote:
>> Why would we want to have two redundant copies of the same information?

> The lock information is not available anywhere in the form of a matrix.

Sure, but at this point we have proposals for adding two different matrix
representations, both redundant with the textual description. I don't
mind adding one of the two, but both seems overkill.

regards, tom lane


From: Peter Eisentraut <peter_e(at)gmx(dot)net>
To: "Simon Riggs" <simon(at)2ndquadrant(dot)com>
Cc: pgsql-patches(at)postgresql(dot)org, "Pavan Deolasee" <pavan(dot)deolasee(at)gmail(dot)com>
Subject: Re: Lock compatibility matrix
Date: 2007-01-30 20:33:26
Message-ID: 200701302133.26825.peter_e@gmx.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-patches

Simon Riggs wrote:
> > Why would we want to have two redundant copies of the same
> > information?
>
> The lock information is not available anywhere in the form of a
> matrix.

But it will be. A patch for the documentation has been proposed.

--
Peter Eisentraut
http://developer.postgresql.org/~petere/


From: "Simon Riggs" <simon(at)2ndquadrant(dot)com>
To: "Peter Eisentraut" <peter_e(at)gmx(dot)net>
Cc: <pgsql-patches(at)postgresql(dot)org>, "Pavan Deolasee" <pavan(dot)deolasee(at)gmail(dot)com>
Subject: Re: Lock compatibility matrix
Date: 2007-01-30 20:45:00
Message-ID: 1170189900.3681.238.camel@silverbirch.site
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-patches

On Tue, 2007-01-30 at 21:33 +0100, Peter Eisentraut wrote:
> Simon Riggs wrote:
> > > Why would we want to have two redundant copies of the same
> > > information?
> >
> > The lock information is not available anywhere in the form of a
> > matrix.
>
> But it will be. A patch for the documentation has been proposed.

Cool. When that's done, we probably don't need the code version.

Would've been helpful if you'd explained what you meant... not many
people read all posts on all lists.

--
Simon Riggs
EnterpriseDB http://www.enterprisedb.com


From: Teodor Sigaev <teodor(at)sigaev(dot)ru>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Simon Riggs <simon(at)2ndquadrant(dot)com>, Peter Eisentraut <peter_e(at)gmx(dot)net>, pgsql-patches(at)postgresql(dot)org, Pavan Deolasee <pavan(dot)deolasee(at)gmail(dot)com>
Subject: Re: Lock compatibility matrix
Date: 2007-01-31 07:45:46
Message-ID: 45C0492A.5030302@sigaev.ru
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-patches

> representations, both redundant with the textual description. I don't
Docs patch is in SGML table representation, text view is a demonstration in mail.

--
Teodor Sigaev E-mail: teodor(at)sigaev(dot)ru
WWW: http://www.sigaev.ru/