public inbox for [email protected]  

help / color / mirror / Atom feed

From: Tom Lane <[email protected]>
To: Greg Stark <[email protected]>
Cc: Gregory Stark (as CFM) <[email protected]>
Cc: Michael Paquier <[email protected]>
Cc: PostgreSQL-development <[email protected]>
Subject: Re: Commitfest 2023-03 starting tomorrow!
Date: Fri, 17 Mar 2023 10:38:40 -0400
Message-ID: <[email protected]> (raw)
In-Reply-To: <CAM-w4HNiKPLV5nq05ZRyvoWAPNhbpudcEMd8ThSv=Ajpnj+njg@mail.gmail.com>
References: <CAM-w4HPjg7NwEWBtXn1empgAg3fqJHifHo_nhgqFWopiYaNxYg@mail.gmail.com>
	<Y/[email protected]>
	<Y//[email protected]>
	<CAM-w4HP16PwRLRjDecvU12T-od-L71nNUjOKAH1CpNou6X7VQw@mail.gmail.com>
	<CAM-w4HORg6G_5nFt4rJLpnuYkwWUh4-hA2OquTS87t8XbPiEbg@mail.gmail.com>
	<CAM-w4HNiKPLV5nq05ZRyvoWAPNhbpudcEMd8ThSv=Ajpnj+njg@mail.gmail.com>

Greg Stark <[email protected]> writes:
>> These patches that are "Needs Review" and have received no comments at
>> all since before March 1st are these.

Just a couple of comments on ones that caught my eye:

>> * Simplify find_my_exec by using realpath(3)

The problem with this one is that Peter would like it to do something
other than what I think it should do.  Not sure how to resolve that.

>> * Fix assertion failure with next_phase_at in snapbuild.c

This one, and others that are bug fixes, probably deserve more slack.

>> * Periodic burst growth of the checkpoint_req counter on replica.

There is recent discussion of this one no?

>> * Fix ParamPathInfo for union-all AppendPath

I pushed this yesterday.

>> * Add OR REPLACE option for CREATE OPERATOR

I think this one should be flat-out rejected.

>> * Partial aggregates push down

You've listed a lot of small features here that still have time to
get some love --- it's not like we're hard up against the end of the CF.
If they'd been in Waiting on Author state for awhile, I'd agree with
booting them, but not when they're in Needs Review.

>> * Set arbitrary GUC options during initdb

I do indeed intend to push this one on my own authority at some point,
but I'm happy to leave it there for now in case anyone wants to take
another look.

>> * Check lateral references within PHVs for memoize cache keys

I think this one is a bug fix too.

>> * Data is copied twice when specifying both child and parent table in
>> publication

Isn't there active discussion of this one?

>> * Improve pg_bsd_indent's handling of multiline initialization expressions

This is going to get pushed, it's just waiting until the commitfest
settles.  I guess you can move it to the next one if you want, but
that won't accomplish much.

			regards, tom lane

view thread (36+ 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: Commitfest 2023-03 starting tomorrow!
  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