Re: commitfest.postgresql.org is no longer fit for purpose
Jacob Champion <jacob.champion@enterprisedb.com>
From: Jacob Champion <jacob.champion@enterprisedb.com>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Melanie Plageman <melanieplageman@gmail.com>,
Robert Haas <robertmhaas@gmail.com>, "pgsql-hackers@postgresql.org" <pgsql-hackers@postgresql.org>
Date: 2024-05-16T21:43:47Z
Lists: pgsql-hackers
On Thu, May 16, 2024 at 2:04 PM Tom Lane <tgl@sss.pgh.pa.us> wrote: > Oh, that's an independent pet peeve of mine. Usually, if I'm > looking over the CF list for a patch to review, I skip over ones > that already show an assigned reviewer, because I don't want to > step on that person's toes. But it seems very common to put > one's name down for review without any immediate intention of > doing work. Or to do a review and wander off, leaving the patch > apparently being tended to but not really. (And I confess I'm > occasionally guilty of both things myself.) Yep, I do the same thing (and have the same pet peeve). > I think it'd be great if we could separate "I'm actively reviewing > this" from "I'm interested in this". As a bonus, adding yourself > to the "interested" list would be a fine proxy for the thumbs-up > or star markers mentioned upthread. > > If those were separate columns, we could implement some sort of > aging scheme whereby somebody who'd not commented for (say) > a week or two would get quasi-automatically moved from the "active > reviewer" column to the "interested" column, whereupon it wouldn't > be impolite for someone else to sign up for active review. +1! --Jacob