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

On Sat, 3 Apr 2021 at 09:47, Andres Freund <andres@anarazel.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