| From: | Greg Sabino Mullane <htamfids(at)gmail(dot)com> |
|---|---|
| To: | pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Finding worthy commitfest entries |
| Date: | 2026-09-14 15:09:10 |
| Message-ID: | CAKAnmm+WrCTKupt=7SOF0p_N6vaTXJ0BwvWYtzhhY5kpEu5sgQ@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
I was looking through the PG20-2 commitfest this morning
https://commitfest.postgresql.org/61/
I had some spare cycles, so I was trying to see if there were any patches
that needed a review (and a reviewer). However, it's not really possible
to gauge that, as not all patches have the "Reviewers" field populated, and
there is also no way to tell how "active" a patch is. I'm going to
pick on two random ones for examples, that both show up when you hit the
"No reviewers" button:
1. https://commitfest.postgresql.org/patch/6689/
"Extensible sync handler registration"
This one has three posts, all from the author, and clearly is in need of a
reviewer. This one is an excellent example of what I was looking for.
2. https://commitfest.postgresql.org/patch/6652/
"Resizable shared memory structures"
This one looks the same as 6689 on the commitfest: one author, and no
reviewer. But the actual email chain is extraordinarily long and vibrant,
with lots of people weighing in. Pretty much the complete opposite of what
I am looking for, as I'm not going to spend hours reading that thread, when
many
people smarter than me have already weighed in. Yet, without clicking the
link to the archives, there is no way to distinguish the two patches.
Some ideas:
* More peer pressure to have people populate the Reviewer field.
Maybe not 'peer' so much as the commitfest manager. :)
* Let logged-in users add other people as reviewers. If I'm already opening
up the archived discussion, I could easily throw some names in as reviewers
from the thread. Then I can easily skip it next time.
* Have the main page show the number of emails in the thread.
* Have the main page show the number of distinct people in the thread.
* Have LLM give it a score and/or text summary, indicating how active it
is, and the likelihood of it needing a reviewer.
* Allow the author (maybe others) to add a flag such as "no reviewer yet"
or "help needed"
Other ideas are welcome. My primary goal is to be able to find and help out
"lonely" patches that need a review, without opening up every single thread.
Cheers,
Greg
| From | Date | Subject | |
|---|---|---|---|
| Next Message | shihao zhong | 2026-09-14 15:13:09 | Re: Correct documentation for protocol version |
| Previous Message | Greg Burd | 2026-09-14 15:08:48 | Re: ExecForceStoreHeapTuple() loses tts_tid, so ORDER BY-op index scans project an invalid ctid |