Re: Path to unreverting "Allow planner to use Merge Append to efficiently implement UNION"

From: Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>
To: David Rowley <dgrowleyml(at)gmail(dot)com>
Cc: PostgreSQL Developers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Path to unreverting "Allow planner to use Merge Append to efficiently implement UNION"
Date: 2024-05-21 12:34:56
Message-ID: 202405211234.k2e47lxv3tsu@alvherre.pgsql
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 2024-May-21, David Rowley wrote:

> I've attached 2 patches.
>
> 0001 is a simple revert of Tom's revert (7204f3591).
> 0002 fixes the issue reported by Hubert.

I would like to request that you don't keep 0001's message as you have
it here. It'd be more readable to take 66c0185a3d14's whole commit
message with a small suffix like "try 2" in the commit title, and add an
additional second paragraph stating it was transiently reverted by
7204f35919b7. Otherwise it's harder to make sense of the commit on its
own later.

--
Álvaro Herrera Breisgau, Deutschland — https://www.EnterpriseDB.com/

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Guillaume Lelarge 2024-05-21 12:43:30 Re: Ambiguous description on new columns
Previous Message Robert Haas 2024-05-21 12:32:56 Re: libpq compression (part 3)