On Fri, Feb 17, 2023 at 09:36:27AM +0100, Drouvot, Bertrand wrote: > On 2/17/23 3:35 AM, Michael Paquier wrote: >> 0001 looks quite committable at this stage, and that's independent on >> the rest. At the end this patch creates four new test files that are >> extended in the next patches: utility, planning, track and cleanup. > > Thanks! LGTM.
Thanks. I have applied the set of regression tests in 0001 and 0002. Note that I have changed the order of the attributes when querying pg_stat_statements, to make easier to follow the diffs generated by the normalization. The unaligned mode would be another option, but it makes not much sense as long as there are no more than two fields with variable lengths. Some extra notes about that: - Should the test for the validation WAL generation metrics be moved out? I am not sure that it makes much sense to separate it as it has a short purpose. - Same issue with user activity, which creates a few roles and makes sure that their activity is tracked? We don't look at the userid in this case, which does not make much sense to me. - Same issue with locking clauses, worth a file of their own? The main file is still named pg_stat_statements.sql, perhaps it should be renamed to something more generic, like general.sql? Or perhaps we could just split the main file with a select.sql (with locking clauses) and a dml.sql? >> I am wondering if others have an opinion to share about that, but, >> yes, 0004 seems enough to begin with. We could always study more >> normalization areas in future releases, taking it slowly. > > Agree. These last ones are staying around for a few more weeks, until the middle of the next CF, I guess. After all this is done, the final changes are very short, showing the effects of the normalization, as of: 6 files changed, 45 insertions(+), 35 deletions(-) -- Michael
signature.asc
Description: PGP signature