Re: Include patch id in commitfest page - Mailing list pgsql-hackers

From Magnus Hagander
Subject Re: Include patch id in commitfest page
Date
Msg-id CABUevEyd40TxEfAG3m_b1-qOt+YK0a8wPrwCUR_aTqkOm23xFA@mail.gmail.com
Whole thread Raw
In response to Re: Include patch id in commitfest page  (Peter Eisentraut <peter@eisentraut.org>)
List pgsql-hackers
On Sat, Jan 11, 2025 at 10:40 AM Peter Eisentraut <peter@eisentraut.org> wrote:
I want to register that I'm not a fan of this change:

https://git.postgresql.org/gitweb/?p=pgcommitfest2.git;a=commit;h=eee60a5332ac0745071086477ff4e189b080d3ab

Rationale: It puts the least important information first.  For the most
part, the CF entry number is irrelevant, so it shouldn't be shown in
such a prominent position.

Maybe

    <a>title here</a> (number here)

would be a compromise.  But looking at the original argument:

"""
Clicking the patch header would sort by the patch ID, but that ID would
not actually be visible. Effectively resulting in random ordering for
the user, so let's just show the ID.
"""

I would suggest that the actual fix would be to order by title instead
of ID.

Or just turn the sorting off.


I'll revert this one pending further discussions, and then we can figure out what we want to put back.

--

pgsql-hackers by date:

Previous
From: Junwang Zhao
Date:
Subject: Re: Some ExecSeqScan optimizations
Next
From: Andrew Dunstan
Date:
Subject: Re: why there is not VACUUM FULL CONCURRENTLY?