OSDN Git Service

Back out patch:
authorBruce Momjian <bruce@momjian.us>
Sat, 4 Jun 2005 22:57:22 +0000 (22:57 +0000)
committerBruce Momjian <bruce@momjian.us>
Sat, 4 Jun 2005 22:57:22 +0000 (22:57 +0000)
commit24af7d4b68ea5cf08c458ffb00fffacc8c6b7d2c
treedde9346f72c7a9909962ccc56014e5cab2496c36
parent954108f92d5477a4527380c9f867bc9401880975
Back out patch:

---------------------------------------------------------------------------

Tom Lane <tgl@sss.pgh.pa.us> writes:
> a_ogawa <a_ogawa@hi-ho.ne.jp> writes:
> > It is a reasonable idea. However, the majority part of MemSet was not
> > able to be avoided by this idea. Because the per-tuple contexts are used
> > at the early stage of executor.
>
> Drat.  Well, what about changing that?  We could introduce additional
> contexts or change the startup behavior so that the ones that are
> frequently reset don't have any data in them unless you are working
> with pass-by-ref values inside the inner loop.

That might be possible. However, I think that we should change only
aset.c about this article.
I thought further: We can check whether context was used from the last
reset even when blocks list is not empty. Please see attached patch.
src/backend/utils/mmgr/aset.c