Re: Add has_large_object_privilege function

On Thu, 12 Sep 2024 19:51:33 +0900
Yugo NAGATA <nagata@sraoss.co.jp> wrote:

> On Thu, 12 Sep 2024 19:07:22 +0900
> Fujii Masao <masao.fujii@oss.nttdata.com> wrote:
> 
> > 
> > 
> > On 2024/09/10 17:45, Yugo NAGATA wrote:
> > > On Mon, 9 Sep 2024 22:59:34 +0900
> > > Fujii Masao <masao.fujii@oss.nttdata.com> wrote:
> > > 
> > >>
> > >>
> > >> On 2024/07/02 16:34, Yugo NAGATA wrote:
> > >>> So, I would like to propose to add
> > >>> has_large_object_function for checking if a user has the privilege on a large
> > >>> object.
> > >>
> > >> +1
> > > 
> > > Thank you for your looking into this!
> > > I've attached a updated patch.
> > 
> > Thanks for updating the patches! LGTM, except for a couple of minor things:
> > 
> > +okui chiba * has_largeobject_privilege_id
> > 
> > "okui chiba" seems to be a garbage text accidentally added.
> > 
> > + *        role by Oid, large object by id, and privileges as AclMode.
> > 
> > "Oid" seems better than "id" in "large object by id".
> 
> Thank you for pointing out it.
> I've fixed them and attached the updated patch, v3.

I confirmed the patches are committed in the master branch.
Thank you!

I've updated the commitfest status to "committed".

Regards,
Yugo Nagata

-- 
Yugo Nagata <nagata@sraoss.co.jp>