Re: Incremental sort for access method with ordered scan support (amcanorderbyop) - Mailing list pgsql-hackers

From Miroslav Bendik
Subject Re: Incremental sort for access method with ordered scan support (amcanorderbyop)
Date
Msg-id CAPoEpV2vgyQTJhmqVqdYSPRrjBNxM+qZ1Xxzepb0V-mCijBARg@mail.gmail.com
Whole thread Raw
In response to Re: Incremental sort for access method with ordered scan support (amcanorderbyop)  (Richard Guo <guofenglinux@gmail.com>)
Responses Re: Incremental sort for access method with ordered scan support (amcanorderbyop)
List pgsql-hackers
Thanks, for suggestions.

On Sun 02. 07. 2023 at 10:18 Richard Guo <guofenglinux@gmail.com> wrote:
> 1. For comment "On success, the result list is ordered by pathkeys.", I
> think it'd be more accurate if we say something like "On success, the
> result list is ordered by pathkeys or a prefix list of pathkeys."
> considering the possibility of incremental sort.
>
> 2. The comment below is not true anymore.
>
>    /*
>     * Note: for any failure to match, we just return NIL immediately.
>     * There is no value in matching just some of the pathkeys.
>     */
> We should either remove it or change it to emphasize that we may return
> a prefix of the pathkeys for incremental sort.

Comments are updated now.

> BTW, would you please add the patch to the CF to not lose track of it?

Submitted <https://commitfest.postgresql.org/43/4433/>

-- 
Best regards
Miroslav

Attachment

pgsql-hackers by date:

Previous
From: jian he
Date:
Subject: Re: Incremental View Maintenance, take 2
Next
From: Noah Misch
Date:
Subject: Re: Should CSV parsing be stricter about mid-field quotes?