Re: Remove unnecessary relabel stripping
Tomas Vondra <tomas.vondra@2ndquadrant.com>
From: Tomas Vondra <tomas.vondra@2ndquadrant.com>
To: Michael Paquier <michael@paquier.xyz>
Cc: Richard Guo <guofenglinux@gmail.com>, PostgreSQL-development <pgsql-hackers@postgresql.org>
Date: 2020-05-01T23:51:34Z
Lists: pgsql-hackers
On Thu, Apr 30, 2020 at 01:40:11PM +0900, Michael Paquier wrote: >On Thu, Apr 30, 2020 at 09:37:41AM +0800, Richard Guo wrote: >> On Thu, Apr 30, 2020 at 8:11 AM Tomas Vondra <tomas.vondra@2ndquadrant.com> >> wrote:> >>> FWIW it'd be better to send the patch to the original thread instead of >>> starting a new one. >> >> Ah yes, you're right. Sorry for not doing so. > >FWIW, I don't find the move from Richard completely incorrect either >as the original thread discusses about a crash in incremental sorts >with sqlsmith, and here we have a patch to remove a useless operation. >Different threads pointing to different issues help to attract >sometimes a more correct audience. Possibly. I agree it's not an entirely clear case. Anyway, I've pushed the fix. regards -- Tomas Vondra http://www.2ndQuadrant.com PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
Commits
-
Simplify cost_incremental_sort a bit
- 60fbb4d76250 13.0 landed