Re: Improve pg_stat_statements by making jumble handle savepoint names better

 

Yes, it should. I had some trouble getting it to work that way in the first place, but now I realize it was just my unfamiliarity with this part of the code. So thanks for the hint: v2 of the patch is much simplified by adding two attributes to theTransactionStmt node. I've also added some tests per your suggestion.

Unrelated to this patch, I'm struggling with meson testing. Why doesn't this update the postgres test binary?:

meson test --suite pg_stat_statements

It runs "ninja" as expected, but it does not put a new 

build/tmp_install/home/greg/pg/17/bin/postgres in place until I do a "meson test"

Cheers,

Greg