Hi Teodor,
Thanks a lot for a review!
> > step1 In pgstat_report_stat() you remove one by one entries from hash and
> > remove them all. Isn't it better to hash_destroy/hash_create or even let hash
> > lives in separate memory context and just resets it?
Agree, fixed.
> > step1 Again, pgstat_report_stat(), all-zero entries aren't deleted from hash
> > although they will be free from point of view of pgStatTabList.
Good point! Fixed.
--
Best regards,
Aleksander Alekseev