On Thu, 3 Jun 2021 at 14:36, Justin Pryzby <pryzby@telsasoft.com> wrote:
>
> You started the thread about what to call the node, but what about its GUC?
>
> Should it be enable_result_cache instead of enable_resultcache?
>
> See also Robert's opinion last year about enable_incrementalsort and "economizing on underscores".
>
https://www.postgresql.org/message-id/flat/CA%2BTgmoazhPwebpOwNbHt1vJoos1eLYhJVQPka%2BpptSLgS685aA%40mail.gmail.com#e6d5ea0a10384d540b924688c6357b21
If starting fresh, either is ok for me. We already have a mix and we
ended up adding the underscore to enable_incremental_sort as people
thought it was more clear. I did vote to keep it as it was without the
underscore as it seemed to follow the other enable* GUCs more closely.
I get the idea that we changed enable_incrementalsort because the word
incremental was fairly long when you compare it to things like hash,
nest and merge. "result" is not too much longer so I don't really
feel like the name enable_resultcache is wrong. Since it's already
called that, I'd rather not confuse things.
To be honest, I'd rather the discussion was about whether we want to
actually call the entire node "Result Cache" in the first place. I
did do an internet search a few weeks ago on "Result Cache" just to
see if anyone was talking about it anywhere that I might miss. I
discovered that Oracle has something of the same name, but going by
[1] it appears that function is for caching results between queries.
That's obviously not what PostgreSQL's Result Cache does. After
learning that, I dislike the name "Result Cache" even more.
FWIW, I'm still willing to go and do the renaming work for the node,
but the time we have to do that is ticking away fast and so far
there's no consensus [2] on any other name. Just a few people
suggesting other names.
David
[1] https://logicalread.com/huge-performance-gains-using-oracle-11g-result-cache-dr01/#.YLhEjLczb9Q
[2] https://www.postgresql.org/message-id/CAApHDvq=yQXr5kqhRviT2RhNKwToaWr9JAN5t+5_PzhuRJ3wvg@mail.gmail.com
