pgsql: Add WaitForLockers in lmgr, refactoring index.c code

Started by Alvaro Herreraover 12 years ago4 messagescomitters
Jump to latest
#1Alvaro Herrera
alvherre@2ndquadrant.com

Add WaitForLockers in lmgr, refactoring index.c code

This is in support of a future REINDEX CONCURRENTLY feature.

Michael Paquier

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/15732b34e8c856bf94469b78ab49446430a9c203

Modified Files
--------------
src/backend/catalog/index.c | 30 +++--------------
src/backend/commands/indexcmds.c | 30 +++--------------
src/backend/storage/lmgr/lmgr.c | 67 ++++++++++++++++++++++++++++++++++++++
src/include/storage/lmgr.h | 4 +++
4 files changed, 81 insertions(+), 50 deletions(-)

--
Sent via pgsql-committers mailing list (pgsql-committers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-committers

#2Andres Freund
andres@anarazel.de
In reply to: Alvaro Herrera (#1)
Re: pgsql: Add WaitForLockers in lmgr, refactoring index.c code

Hi,

On 2013-10-01 21:00:43 +0000, Alvaro Herrera wrote:

Add WaitForLockers in lmgr, refactoring index.c code

This is in support of a future REINDEX CONCURRENTLY feature.

I think this removes too many comments from index_drop that explains why
it's done that way.

Greetings,

Andres Freund

--
Andres Freund http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services

--
Sent via pgsql-committers mailing list (pgsql-committers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-committers

#3Alvaro Herrera
alvherre@2ndquadrant.com
In reply to: Andres Freund (#2)
Re: pgsql: Add WaitForLockers in lmgr, refactoring index.c code

Andres Freund wrote:

Hi,

On 2013-10-01 21:00:43 +0000, Alvaro Herrera wrote:

Add WaitForLockers in lmgr, refactoring index.c code

This is in support of a future REINDEX CONCURRENTLY feature.

I think this removes too many comments from index_drop that explains why
it's done that way.

Complaint registered. If either of you can send a patch I will gladly
push it, otherwise it will take me some time to get back to it (but will
eventually).

--
�lvaro Herrera http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services

--
Sent via pgsql-committers mailing list (pgsql-committers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-committers

#4Michael Paquier
michael@paquier.xyz
In reply to: Alvaro Herrera (#3)
Re: pgsql: Add WaitForLockers in lmgr, refactoring index.c code

On Wed, Oct 2, 2013 at 6:32 AM, Alvaro Herrera <alvherre@2ndquadrant.com> wrote:

Andres Freund wrote:

I think this removes too many comments from index_drop that explains why
it's done that way.

Complaint registered. If either of you can send a patch I will gladly
push it, otherwise it will take me some time to get back to it (but will
eventually).

Indeed, it might be better to add some more comments. Attached is a
patch improving that.
Regards,
--
Michael

Attachments:

20131002_index_drop_comments.patchapplication/octet-stream; name=20131002_index_drop_comments.patchDownload+9-6