Wednesday, August 27, 2008

Re: [HACKERS] Another refactoring proposal: move stuff into nodes/nodeFuncs.[ch]

Simon Riggs <simon@2ndQuadrant.com> writes:
> On Mon, 2008-08-25 at 11:24 -0400, Tom Lane wrote:
>> The advantages of doing this would be (a) reduce the number of places
>> to look in when implementing a new node type; (b) eliminate some
>> cross-subsystem #inclusions that weaken modularity of the backend.

> Are we doing either of those things in this release?

Yes, it's done already.

> Might these changes screw up patches already in progress? Can we hold
> off making these changes until we're sure the latter isn't true?

Some patches might need small adjustments (to find the code in a
different file) but that does not strike me as an argument for not
changing things. We have applied far more invasive patches in the past
and undoubtedly will do so again in future. In any case, it's still a
long way until beta freeze, so there's plenty of time to deal with
fallout. I don't see that some other time would be better.

regards, tom lane

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

No comments: