On 2021-Oct-16, Tom Lane wrote: > Attached is a proposed patch that refactors getTables() along the > same lines as some previous work (eg 047329624, ed2c7f65b, daa9fe8a5) > to avoid having multiple partially-redundant copies of the SQL query. > This gets rid of nearly 300 lines of duplicative spaghetti code, > creates a uniform style for dealing with cross-version changes > (replacing at least three different methods currently being used > for that in this same stretch of code), and allows moving some > comments to be closer to the code they describe.
Yeah, this seems a lot better than the original coding. Maybe I would group together the changes that all require the same version test, rather than keeping the output columns in the same order. This reduces the number of branches. Because the follow-on code uses column names rather than numbers, there is no reason to keep related columns together. But it's a clear improvement even without that. -- Álvaro Herrera 39°49'30"S 73°17'W — https://www.EnterpriseDB.com/ "Puedes vivir sólo una vez, pero si lo haces bien, una vez es suficiente"
