Re: Non-committer reviews: is it helpful? - Mailing list pgsql-hackers

From David G. Johnston
Subject Re: Non-committer reviews: is it helpful?
Date
Msg-id CAKFQuwba2iFVyLb3+Jab5ruWdPxNqBCFVFMDeCdmT_hVUYm43w@mail.gmail.com
Whole thread Raw
In response to Non-committer reviews: is it helpful?  (Alexander Borisov <lex.borisov@gmail.com>)
Responses Re: Non-committer reviews: is it helpful?
List pgsql-hackers
On Thu, Jan 29, 2026 at 3:07 PM Alexander Borisov <lex.borisov@gmail.com> wrote:
I have a question for the community. While waiting for feedback on my
own patches, I was wondering: is there value in me reviewing other
people's patches, even though I am not a committer here?


Non-committers moving patches from "needs review" to "ready for committer" is a critical activity here.  It is part of the soft contract one enters into when authoring patches - to try and perform an amount of review work equivalent to what you are asking from others.

As for which ones to review - everyone has their own natural prioritization methodology (and capabilities) just by interacting with the project and community.  I'd suggest you don't overthink things and just do what feels right to you.

David J.

pgsql-hackers by date:

Previous
From: Nathan Bossart
Date:
Subject: Re: split tablecmds.c
Next
From: Sami Imseih
Date:
Subject: Re: Optional skipping of unchanged relations during ANALYZE?