Re: Refactoring xlogutils.c
Tom Lane <[email protected]>
| Newsgroups | gmane.comp.db.postgresql.devel.patches |
|---|---|
| Message-ID | <[email protected]> |
"Heikki Linnakangas" <[email protected]> writes: > Attached is an updated version of my patch to refactor the > XLogOpenRelation/XLogReadBuffer interface, in preparation for the > relation forks patch, and subsequently the FSM rewrite patch. The code motion in md.c looks fairly bogus; was that a copy-and-paste error? Otherwise it looks pretty sane, but I have one stylistic gripe: I'm dubious about your willingness to assume that pfree() is enough for getting rid of a fake relcache entry. Relcache entries are complex beasts and it may not continue to work to do that. It's also possible that we'd have additional resources attached to them someday. So I think it'd be worth having a FreeFakeRelcacheEntry() routine to localize the knowledge of how to get rid of them. regards, tom lane -- Sent via pgsql-patches mailing list ([email protected]) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-patches