Re: pgsql: Add Result Cache executor node (take 2)

On Sat, 3 Apr 2021 at 09:47, Andres Freund <andres(at)anarazel(dot)de> wrote:
> Since this commit I am seeing
>
> In file included from /home/andres/src/postgresql/src/include/postgres.h:46,
> from /home/andres/src/postgresql/src/backend/executor/nodeResultCache.c:67:
> /home/andres/src/postgresql/src/backend/executor/nodeResultCache.c: In function ‘entry_purge_tuples’:
> /home/andres/src/postgresql/src/backend/executor/nodeResultCache.c:290:27: warning: comparison of unsigned expression in ‘>= 0’ is always true [-Wtype-limits]
> 290 | Assert(rcstate->mem_used >= 0);
> | ^~

Those were last-minute additions and seems not well thought through.
I've now removed them.

Thanks for the report.

David