Melanie Plageman <melanieplageman@gmail.com> writes:
> So, while I agree that the "Single" in SetSingleFuncCall() could be
> confusing given the name of ExprSingleResult, I feel like actually all
> of the names are somewhat wrong.
Maybe, but ExprSingleResult et al. have been there for decades and
are certainly embedded in a ton of third-party code. It's a bit
late to rename them, whether you think they're confusing or not.
Maybe we can get away with changing names introduced in v15, but
even that I'm afraid will get some pushback.
Having said that, I'd probably have used names based on "materialize"
not "single" for what this code is doing.
regards, tom lane