file portability/instr_time.h, and add a couple more macros to eliminate
some abstraction leakage we formerly had.  Also update psql to use this
header instead of its own copy of nearly the same code.

This commit in itself is just code cleanup and shouldn't change anything.
It lays some groundwork for the upcoming function-stats patch, though.

back-stamped for this.

because node timing is much less predictable than the patch expects.  I kept
the API change for InstrStopNode, however.

across multiple loops, get rid of the shaky assumption that exactly one
tuple is returned per node iteration.

Martijn van Oosterhout

return just a single tuple at a time.  Currently the only such node
type is Hash, but I expect we will soon have indexscans that can return
tuple bitmaps.  A side benefit is that EXPLAIN ANALYZE now shows the
correct tuple count for a Hash node.

executing a statement that fires triggers.  Formerly this time was
included in "Total runtime" but not otherwise accounted for.
As a side benefit, we avoid re-opening relations when firing non-deferred
AFTER triggers, because the trigger code can re-use the main executor's
ResultRelInfo data structure.

EXPLAIN ANALYZE instrumentation.  Magnus Hagander

make sure the $Id tags are converted to $PostgreSQL as well ...

so it won't miss 'em again.

initdb/regression tests pass.

spacing.  Also adds space for one-line comments.

tests pass.

counts alongside the planner's estimates.  By Martijn van Oosterhout,
with some further work by Tom Lane.