Re: moving some code out of explain.c
Peter Geoghegan <pg@bowt.ie>
From: Peter Geoghegan <pg@bowt.ie>
To: Robert Haas <robertmhaas@gmail.com>
Cc: PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2025-02-18T18:11:23Z
Lists: pgsql-hackers
On Tue, Feb 18, 2025 at 1:04 PM Robert Haas <robertmhaas@gmail.com> wrote: > I think in cases like this there is a tendency to want to achieve an > even split of the original file, but in practice I think that tends > not to be difficult to achieve. These two patches combined move about > 1000 lines of code into separate files, which I think is actually > quite a good result for a refactoring of this sort. We might want to > split it up even more, but I don't feel like that has to be done in > the first set of patches or that all such patches have to come from > me. So I propose to do just this much for now. > > Comments? Seems like a good idea to me. -- Peter Geoghegan
Commits
-
Avoid including explain.h in explain_format.h and explain_dr.h
- 77cb08be5106 18.0 landed
-
Create explain_dr.c and move DestReceiver-related code there.
- 555960a0fbf0 18.0 landed
-
Create explain_format.c and move relevant code there.
- 9173e8b60463 18.0 landed