"Alex Hunsaker" <badalex@gmail.com> wrote:
> I was assigned to review this.
Thanks for your reviewing.
I assume that the basic concepts are ok and focus of discussion is in: - New counters in struct Instrumentation.
(bufferusage and CPU usage) - Should EXPLAIN ANALYZE show those counters.
> Performance review
> HEAD: tps = 9.674423
> PATCH: tps = 9.695784
>
> If it claims to improve performance, does it?
> Does it slow down other things?
The patch should not slow down normal use if you don't use
pg_stat_statements module, but it might slow down EXPLAIN ANALYZE
because some fields are added in struct Instrumentation and
they are counted up per tuple in EXPLAIN ANALYZE.
> Also find attached some very minor verbiage changes.
Thanks. I'll apply your fixes.
Regards,
---
ITAGAKI Takahiro
NTT Open Source Software Center