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

From Peter Eisentraut
Subject Re: Include patch id in commitfest page
Date
Msg-id ff9d7921-f4e3-498a-9dba-ddd28fdb359f@eisentraut.org
Whole thread Raw
Responses Re: Include patch id in commitfest page
Re: Include patch id in commitfest page
List pgsql-hackers
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.




pgsql-hackers by date:

Previous
From: yuansong
Date:
Subject: Re:Re: Re:Re:Re: backup server core when redo btree_xlog_insert that type is XLOG_BTREE_INSERT_POST
Next
From: Kirill Reshke
Date:
Subject: Re: Change COPY ... ON_ERROR ignore to ON_ERROR ignore_row