public inbox for [email protected]  
help / color / mirror / Atom feed
From: Bruce Momjian <[email protected]>
To: Nazir Bilal Yavuz <[email protected]>
Cc: PostgreSQL-development <[email protected]>
Cc: Thomas Munro <[email protected]>
Subject: Re: PG 18 release notes draft committed
Date: Tue, 20 May 2025 09:52:17 -0400
Message-ID: <[email protected]> (raw)
In-Reply-To: <CAN55FZ0ZaXUpEjO1WB7zpxVFKZEphVWQJnUZXVkMQNzpjseo2Q@mail.gmail.com>
References: <[email protected]>
	<CAN55FZ0ZaXUpEjO1WB7zpxVFKZEphVWQJnUZXVkMQNzpjseo2Q@mail.gmail.com>

On Tue, May 20, 2025 at 03:46:44PM +0300, Nazir Bilal Yavuz wrote:
> Hi,
> 
> Thanks for working on this!
> 
> On Fri, 2 May 2025 at 05:44, Bruce Momjian <[email protected]> wrote:
> >
> > I will continue improving it until beta 1, and until the final release.
> > I will probably add markup in 1-3 weeks.  Let the feedback begin.  ;-)
> 
> +<listitem>
> +<para>
> +Add server variable file_copy_method to control the file copying
> method (Nazir Bilal Yavuz)
> +<ulink url="&commit_baseurl;f78ca6f3e">&sect;</ulink>
> +</para>

Uh, the commit is:

	commit f78ca6f3ebb
	Author: Thomas Munro <[email protected]>
	Date:   Tue Apr 8 20:52:47 2025 +1200
	
	    Introduce file_copy_method setting.
	
	    It can be set to either COPY (the default) or CLONE if the system
	    supports it.  CLONE causes callers of copydir(), currently CREATE
	    DATABASE ... STRATEGY=FILE_COPY and ALTER DATABASE ... SET TABLESPACE =
	    ..., to use copy_file_range (Linux, FreeBSD) or copyfile (macOS) to copy
	    files instead of a read-write loop over the contents.
	
	    CLONE gives the kernel the opportunity to share block ranges on
	    copy-on-write file systems and push copying down to storage on others,
	    depending on configuration.  On some systems CLONE can be used to clone
	    large databases quickly with CREATE DATABASE ... TEMPLATE=source
	    STRATEGY=FILE_COPY.
	
	    Other operating systems could be supported; patches welcome.
	
	    Co-authored-by: Nazir Bilal Yavuz <[email protected]>
	    Reviewed-by: Robert Haas <[email protected]>
	    Reviewed-by: Ranier Vilela <[email protected]>
	    Discussion: https://postgr.es/m/CA%2BhUKGLM%2Bt%2BSwBU-cHeMUXJCOgBxSHLGZutV5zCwY4qrCcE02w%40mail.gmail.com

As of the date of the commit, "Co-authored-by:" is listed as:

	https://wiki.postgresql.org/wiki/Commit_Message_Guidance

	"Co-authored-by:" is used by committers when they want to give full credit
	to the named individuals, but also indicate that they made significant
	changes.

> A minor fix; I co-authored this with Thomas Munro, he is the actual author.

Uh, does this mean I should add Thomas Munro before or after your name,
or remove your name and list only Thomas Munro?

-- 
  Bruce Momjian  <[email protected]>        https://momjian.us
  EDB                                      https://enterprisedb.com

  Do not let urgent matters crowd out time for investment in the future.





view thread (147+ 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: PG 18 release notes draft committed
  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