On 2020-Sep-18, Michael Paquier wrote: > Based on the low level of activity and the fact that the patch was > marked as waiting on author for a couple of weeks, it looks like > little could be achieved by the end of the CF, and the attention was > elsewhere, so it looked better (and it still does IMO) to give more > attention to the remaining 170~ patches that are still lying on the CF > app.
"A couple of weeks" of inactivity is not sufficient, in my view, to boot a patch out of the commitfest process. Whenever the patch is resurrected, it will be a new entry which won't have the history that it had accumulated in the long time since it was created -- which biases it against other new patches. I'm not really arguing about this one patch only, but more generally about the process you're following. The problem is that you as CFM are imposing your personal priorities on the whole process by punting patches ahead of time -- you are saying that nobody else should be looking at updating this patch because it is now closed. It seems fair to do so at the end of the commitfest, but it does not seem fair to do it when it's still mid-cycle. Putting also in perspective the history that the patch had prior to your unfairly closing it, there was a lot of effort in getting it reviewed to a good state and updated by the author -- yet a single unsubstantiated comment, without itself a lot of effort on the reviewer's part, that "maybe it has a perf drawback" is sufficient to get it booted? It doesn't seem appropriate. -- Álvaro Herrera https://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services