Replies: 2 comments
|
Here is some background about this discussion. Recently @carloea2 created a large number of PRs concurrently. People expressed at least two concerns:
So it will be better to raise these PRs in small batches. After seeing this discussion, a few members, including @carloea2 @aglinxinyuan @Xiao-zhen-Liu and me (@chenlica ) had an offline meeting and explained these concerns. @carloea2 Please share your thoughts after the meeting. Thanks. |
|
Thanks. I agree with the concerns you summarized. The offline discussion was helpful, and I understand that opening many PRs concurrently can put pressure on CI resources and make review feedback harder to apply consistently across related changes. Going forward, I’ll submit changes in smaller batches and coordinate when I have several related PRs ready. I’ll also give existing PRs time to move through review before opening another large batch. Thanks everyone for the feedback and for taking the time to discuss this. |
Uh oh!
There was an error while loading. Please reload this page.
What pace of PR submissions can the community comfortably support? Small, focused PRs help with individual reviews, but opening many at once also creates more requests for reviewers to track.
To make this discussion concrete, here are statistics for PRs opened in the 90 days ending September 23, 2026. There were 1,303 human-authored PRs from 40 contributors, excluding 186 automated PRs.
Submission rate
The overall rate decreased across the last two 30-day windows: 498 → 351 PRs. However, averages hide bursts. My own submissions include 58 PRs opened on August 31; 55 of my open, non-draft PRs had no submitted human review when checked.
PR sizes also differ. The median size of my PRs was 17 added/deleted lines, compared with approximately 228 for aglinxinyuan and 161 for mengw15. A PR count alone cannot describe the effort needed to review them.
Who handles the reviews?
Related to #8431, review work is concentrated among a few people.
The top two account for 56% of reviewer–PR relationships and 73% of merges. The top four account for 82% of reviewer–PR relationships.
The author–reviewer relationships also show how much particular contributors depend on particular reviewers:
What is waiting?
Of the human-authored PRs opened during this period:
Among PRs that received a review, the median time to first human review was 8.6 hours. Among merged PRs, the median time to merge was 15.6 hours. These figures exclude unresolved waits, so they should be read alongside the queue numbers.
Questions for the community
The numbers show different submission patterns and concentrated review work, but they do not establish what rate is sustainable. It would be useful to hear from the people doing the reviews about what is manageable.
Source and method: GitHub API metadata for apache/texera, June 25–September 23, 2026, with both boundaries at 23:02 UTC. Weekly averages include inactive weeks. Each human reviewer counts once per PR, excluding self-reviews and bots; ordinary conversation comments are not counted as reviews. Merge actions are counted separately and can include authors merging their own reviewed PRs. Related PRs and backports count separately. Open counts include only PRs created within this window, not the entire repository backlog.
All reactions