Michail Nikolaev <michail.nikol...@gmail.com> writes:
> I have added small change to patch to allow it be compiled using msvc
> (uint64_t -> uint64).
> Everything seems to work, check-world is passing.
> Actually patch fixes two issues:
> 1) Partial GIST indexes now have corrent tuples count estimation.
> 2) Now subsequent calls to VACUUM on GIST index (like "VACCUM table_name") do
> not change tuples count to estimated number of tuples in table (which is
> changed even without any updates in table due current implementation).
> I think it is fine to commit.
I took a quick look at this. I wonder what is the point of making
the counting conditional. Since the function is visiting every
index page anyway, why not just always count and unconditionally
provide an exact answer? The number of cycles saved by skipping
"tuplesCount += PageGetMaxOffsetNumber(page)" on each leaf page
is surely trivial.
regards, tom lane
