Michael Paquier <michael(at)paquier(dot)xyz> writes:
> But the patch enforces the attribute name to be the underlying
> function name, switching the previous "current_catalog" to
> "current_database".
The entire point of SQLValueFunction IMO was to hide the underlying
implementation(s). Replacing it with something that leaks
implementation details does not seem like a step forward.
regards, tom lane