Skip site navigation (1) Skip section navigation (2)

Re: [COMMITTERS] pgsql: Assorted cleanups in preparation for using a map file to support

From: Simon Riggs <simon(at)2ndQuadrant(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: pgsql-hackers(at)postgreSQL(dot)org
Subject: Re: [COMMITTERS] pgsql: Assorted cleanups in preparation for using a map file to support
Date: 2010-02-03 16:21:26
Message-ID: 1265214086.1729.2388.camel@ebony (view raw or flat)
Thread:
Lists: pgsql-committerspgsql-hackers
On Wed, 2010-02-03 at 10:48 -0500, Tom Lane wrote:

> > If so, there is some minor code cleanup and comment changes in
> > ProcessCommittedInvalidationMessages(). Would you like me to do that, or
> > should we wait?
> 
> I saw that.  I didn't touch it because it's not directly relevant to
> what I'm doing right now, but I would like to go back and see whether
> that routine can't be got rid of completely.  It seems to me to be a
> very klugy substitute for having enough information.  I'm inclined to
> think that we should emit an sinval message (or maybe better a separate
> WAL entry) for initfile removal, instead of trying to reverse-engineer
> whether one happened.

An additional sinval message type would work. There is a requirement for
us to run RelationCacheInitFileInvalidate() both before and after the
other messages. So we would need to append and prepend the new message
type onto the array of messages if transInvalInfo->RelcacheInitFileInval
is true. That way we would just do SendSharedInvalidMessages() in
xact_redo_commit and remove ProcessCommittedInvalidationMessages(),
adding other code to handle the inval message. Doesn't seem any easier
though.

Another WAL record would definitely be cumbersome.

-- 
 Simon Riggs           www.2ndQuadrant.com


In response to

Responses

pgsql-hackers by date

Next:From: Tim BunceDate: 2010-02-03 16:31:30
Subject: Re: Add on_trusted_init and on_untrusted_init to plperl UPDATED [PATCH]
Previous:From: Marko TiikkajaDate: 2010-02-03 16:18:17
Subject: Re: Review of Writeable CTE Patch

pgsql-committers by date

Next:From: Bruce MomjianDate: 2010-02-03 17:25:06
Subject: pgsql: Proofreading improvements for the Administration documentation
Previous:From: Tom LaneDate: 2010-02-03 15:48:20
Subject: Re: [COMMITTERS] pgsql: Assorted cleanups in preparation for using a map file to support

Privacy Policy | About PostgreSQL
Copyright © 1996-2014 The PostgreSQL Global Development Group