On Thu, Jul 11, 2024 at 07:10:58PM +0530, Ashutosh Bapat wrote: > I think it's just convenient. We do that at multiple places; not exactly > these functions but functions which fetch relation attributes from cached > tuples. Given that the tuple is cached and local to the backend, it's not > too expensive. But if there are multiple places which do something like > this, we may want to create more function get_rel_* function which return > multiple properties in one function call. I see get_rel_namspace() and > get_rel_name() called together at many places.
That's not worth the complications based on the level of caching. This code is fine as-is. -- Michael
signature.asc
Description: PGP signature
