Michael Paquier <mich...@paquier.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