> Yes, thanks. Structurally, this is better and more flexible than what > we had originally, and I have noticed that you have copied the > original files while adding more comments and renaming a bit things: > the structure of the functions was exactly the same. Anyway, I have > worked on that for a good portion of the day, splitting the module > drop and the new module into two commits, and applied the result after > tweaking quite a few things in terms of names and comments (no > pgstat_*, a bit more "Var" and "Fixed", etc.), applying a much more > consistent set of names across the board for the functions and the > structures. This cleanup part is moved out of the way now, so that > you ease the introduction of the next pieces you are proposing.
Thanks for getting these committed!
I rebased the custom callbacks patch in v5.
One very minor thing from the earlier commits that I corrected here is
the test for entry 2 after a clean restart.
-is($result, "entry1|2", "variable-sized stats persist after clean restart");
+is($result, "entry1|2|Test entry 1", "variable-sized stats persist
after clean restart");
+
+$result = $node->safe_psql('postgres', q(select * from
test_custom_stats_var_report('entry2')));
+is($result, "entry2|3|Test entry 2", "variable-sized stats persist
after clean restart");
+
--
Sami Imseih
Amazon Web Services (AWS)
v5-0001-Allow-cumulative-statistics-to-serialize-auxiliar.patch
Description: Binary data
