public inbox for [email protected]  

help / color / mirror / Atom feed

From: Yugo NAGATA <[email protected]>
To: Laurenz Albe <[email protected]>
Cc: Nathan Bossart <[email protected]>
Cc: Tom Lane <[email protected]>
Cc: [email protected]
Subject: Re: Extend ALTER DEFAULT PRIVILEGES for large objects
Date: Thu, 23 Jan 2025 19:22:20 +0900
Message-ID: <[email protected]> (raw)
In-Reply-To: <[email protected]>
References: <[email protected]>
	<[email protected]>
	<20240424210839.GA863220@nathanxps13>
	<[email protected]>
	<[email protected]>
	<[email protected]>

On Wed, 22 Jan 2025 13:30:17 +0100
Laurenz Albe <[email protected]> wrote:

> On Fri, 2024-09-13 at 16:18 +0900, Yugo Nagata wrote:
> > I've attached a updated patch. The test is rewritten using has_largeobject_privilege()
> > function instead of calling loread & lowrite, which makes the test a bit simpler.
> > Thare are no other changes.
> 
> When I tried to apply this patch, I found that it doesn't apply any
> more since commit f391d9dc93 renamed tab-complete.c to tab-complete.in.c.
> 
> Attached is a rebased patch.

Thank you for updating the patch!

> I agree that large objects are a feature that should fade out (alas,
> the JDBC driver still uses it for BLOBs).  But this patch is not big
> or complicated and is unlikely to create a big maintenance burden.
> 
> So I am somewhat for committing it.  It works as advertised.
> If you are fine with my rebased patch, I can mark it as "ready for
> committer".  If it actually gets committed depends on whether there
> is a committer who thinks it worth the effort or not.

I confirmed the patch and I am fine with it.

Regards,
Yugo Nagata

-- 
Yugo NAGATA <[email protected]>

view thread (21+ messages)  latest in thread

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected], [email protected]
  Subject: Re: Extend ALTER DEFAULT PRIVILEGES for large objects
  In-Reply-To: <[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox