On 2026/10/26 12:06:23 Jan Høydahl wrote:
> Let's give this some more time before adjusting. To prioritize the backlog of
> 90 PRs, you can set every active PR to DRAFT and then select the 7 highest
> prioritized ones for committer review. We do NOT want 100 open PRs in the
> review queue.
>
> Jan
>
> > 26. okt. 2026 kl. 12:52 skrev Eric Pugh <[email protected]>:
> >
> >
> >
> > On 2026/10/26 05:01:44 Jalaz Kumar wrote:
> >> Yes correct, that’s what Jan fixed in the yaml file.
> >>
> >> For now, including draft PRs under this check is not yet deployed from ASF
> >> infra team, that’s why the parsing of yaml file was broken earlier.
> >>
> >> Jan removed that problematic key, post which it’s working partially now,
> >> including drafts counts also in check will be kind of follow up now in
> >> future once its sorted out on infra side.
> >>
> >>
> >> Regards
> >> Jalaz Kumar
> >>
> >> On Mon, 26 Oct 2026 at 10:10 AM, Nick Boss <[email protected]> wrote:
> >>
> >>> The change prevents me from marking Drafts as Open but doesn't prevent me
> >>> from opening a Draft.
> >>> Opening a PR returns 404 unless it's marked Draft.
> >>>
> >>> On Sun, Oct 25, 2026 at 8:03 PM Jan Høydahl <[email protected]> wrote:
> >>>
> >>>> There was an error parsing our asf.yaml, fixed, please try again...
> >>>>
> >>>> Jan
> >>>>
> >>>>> 24. okt. 2026 kl. 23:42 skrev Jalaz Kumar <[email protected]>:
> >>>>>
> >>>>> Hi Jan
> >>>>>
> >>>>> Thanks for adding this cap, I can see that the PR is merged now. But I
> >>>>> don't think it's taking effect.
> >>>>> Out of curiosity, I tested this after the merge and I was able to open
> >>> 7+
> >>>>> PRs since then without any block/warning.
> >>>>>
> >>>>> All 8 are still open, plus an older one. so, 9 are open in total. The
> >>> cap
> >>>>> should have been at 7.
> >>>>>
> >>>>> Is there something more needed at ASF-Infra side for this to take
> >>> effect,
> >>>>> or I may be missing something?
> >>>>>
> >>>>>
> >>>>> On Fri, Oct 16, 2026 at 8:14 PM Jan Høydahl <[email protected]> wrote:
> >>>>>
> >>>>>> https://github.com/apache/solr/pull/5163
> >>>>>>
> >>>>>>> 16. okt. 2026 kl. 16:31 skrev Jan Høydahl <[email protected]>:
> >>>>>>>
> >>>>>>> Hi,
> >>>>>>>
> >>>>>>> So far only positive feedback. Here's the PR, I'll merge on Monday...
> >>>>>>>
> >>>>>>> Jan
> >>>>>>>
> >>>>>>>> 15. okt. 2026 kl. 17:20 skrev Jan Høydahl <[email protected]>:
> >>>>>>>>
> >>>>>>>> Hello,
> >>>>>>>>
> >>>>>>>> The Apache Infrastructure team has recently introduced a
> >>> configuration
> >>>>>> option that allows projects to set a maximum limit on concurrent pull
> >>>>>> requests from external contributors—those without write access—while
> >>>>>> exempting committers. Details are available at
> >>>>>>
> >>>>
> >>> https://github.com/apache/infrastructure-asfyaml#pull-request-creation-cap
> >>>>>> .
> >>>>>>>>
> >>>>>>>> I propose implementing a cap across all Solr repositories. This
> >>>>>> adjustment would better align with our review capacity and encourage
> >>>>>> contributors to prioritize existing submissions.
> >>>>>>>>
> >>>>>>>> I suggest a threshold of seven as an initial estimate. A potential
> >>>> risk
> >>>>>> is that contributors might submit larger pull requests to maximize the
> >>>>>> limit, which could reduce reviewability; however, we can address this
> >>>>>> through feedback.
> >>>>>>>>
> >>>>>>>> This message is not a formal vote, and I do not anticipate one. If
> >>> the
> >>>>>> discussion lacks consensus, becomes contentious, or encounters
> >>>> significant
> >>>>>> resistance, we may initiate a vote to resolve the matter.
> >>>>>>>>
> >>>>>>>> Jan
> >>>>>>>
> >>>>>>
> >>>>>>
> >>>>>
> >>>>> --
> >>>>> Regards
> >>>>> Jalaz Kumar
> >>>>
> >>>>
> >>>> ---------------------------------------------------------------------
> >>>> To unsubscribe, e-mail: [email protected]
> >>>> For additional commands, e-mail: [email protected]
> >>>>
> >>>>
> >>>
> >>
> >
> > So, one side effect... We have a draft pr
> > https://github.com/apache/solr/pull/5230 but I can't merge it..
> >
> > I know we want to push things over the finish line, and I know Nick is an
> > outlier a bit, but 7 may be too low.
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: [email protected]
> > For additional commands, e-mail: [email protected]
> >
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [email protected]
> For additional commands, e-mail: [email protected]
>
>
That is totally a fair point! you don't have to keep your 7 active for
forever, pick the top, makes sense!
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
