I appreciate you fixing this and adding the new circuit breaker and look forward to having it in the hands of our users soon.
However, the current state of PR, with significant API churn for a single change and overlapping code is not yet ready. If this is too much of a rework, I am happy to take the existing PR and do the changes, post which I believe the PR should be close to completion. Let me know if you need me to help, but unfortunately, the two objections I raised are blockers, atleast until we establish that they cannot be done away with. On Wed, 2 Jun 2021, 01:37 Walter Underwood, <wun...@wunderwood.org> wrote: > I would appreciate a second opinion on the pull request. Substantive > issues have been resolved. At this point, the discussion is about code > style and coding standards. I don’t have detailed knowledge about the Solr > coding style, so I’d appreciate another set of eyes. > > The current behavior is buggy, and we are not able to use it at Chegg. The > patch fixes those bugs. > > https://github.com/apache/solr/pull/96 > > wunder > Walter Underwood > wun...@wunderwood.org > http://observer.wunderwood.org/ (my blog) > > On Jun 1, 2021, at 12:27 PM, Walter Underwood <wun...@wunderwood.org> > wrote: > > I answered the comments. I don’t see those answers on github, oddly. > > I’ll re-answer them. Most of your questions are already answered in the > discussion on Jira. > > I central issues is that load average is not always a CPU measure. In some > systems, it includes threads in iowait. So it is potentially misleading to > label it as CPU and document it as CPU. The updated documentation makes > that clear, so that should have already answered your comment. that is why > it is important to rename the existing circuit breaker. > > wunder > Walter Underwood > wun...@wunderwood.org > http://observer.wunderwood.org/ (my blog) > > On Jun 1, 2021, at 12:20 PM, Atri Sharma <a...@apache.org> wrote: > > I tool a look at the PR and gave comments for SOLR-15056, and the last I > checked, my comments were not addressed? > > On Wed, 2 Jun 2021, 00:31 Walter Underwood, <wun...@wunderwood.org> wrote: > >> Could someone else please take a look at SOLR-15056? This is a small >> blast radius change that improves the circuit breakers. It includes unit >> tests and documentation and has been ready since January. >> >> https://github.com/apache/solr/pull/96/files >> https://issues.apache.org/jira/browse/SOLR-15056 >> >> wunder >> Walter Underwood >> wun...@wunderwood.org >> http://observer.wunderwood.org/ (my blog) >> >> On Jun 1, 2021, at 11:53 AM, Mayya Sharipova < >> mayya.sharip...@elastic.co.INVALID> wrote: >> >> Thank you for the update, Houston. >> >> I've started the release process, the branch 8.9 is now cut. >> >> On Tue, Jun 1, 2021 at 11:21 AM Houston Putman <hous...@apache.org> >> wrote: >> >>> Mayya, SOLR-14978 is now in 8.x. So no longer a blocker. >>> >>> - Houston >>> >>> On Thu, May 27, 2021 at 11:42 PM David Smiley <dsmi...@apache.org> >>> wrote: >>> >>>> SOLR-15412 is rather serious as the title suggests. I haven't been >>>> tracking the progress so if it's already resolved, that's unknown to me and >>>> isn't reflected in JIRA. >>>> >>>> ~ David Smiley >>>> Apache Lucene/Solr Search Developer >>>> http://www.linkedin.com/in/davidwsmiley >>>> >>>> >>>> On Thu, May 27, 2021 at 5:24 PM Mayya Sharipova < >>>> mayya.sharip...@elastic.co.invalid> wrote: >>>> >>>>> Hello everyone, >>>>> I wonder if everyone is ok for May 31st (Monday) as the date for the >>>>> feature freeze date and branch cut? >>>>> I've noticed that `releaseWizard.py` is also asking for the length of >>>>> feature freeze. What is the custom length to put there? >>>>> >>>>> Looks like Lucene >>>>> <https://issues.apache.org/jira/projects/LUCENE/versions/12349562> >>>>> doesn't have any unresolved issues for 8.9. >>>>> SOLR <https://issues.apache.org/jira/projects/SOLR/versions/12349563> >>>>> has: >>>>> - SOLR-15412 Strict validation on Replica metadata can cause >>>>> complete outage (Looks like it may be resolved already?) >>>>> - SOLR-15410 GC log is directed to console when starting Solr with >>>>> Java 11 Open J9 on Windows >>>>> - SOLR-15056 CPU circuit breaker needs to use CPU utilization, not >>>>> Unix load average >>>>> >>>>> Are we ok to postpone these issues to later releases if they are not >>>>> resolved and merged before feature freeze? >>>>> >>>>> Thank you. >>>>> >>>>> >>>>> >>>>> >>>>> >>>>> >>>>> On Tue, May 25, 2021 at 12:41 PM Colvin Cowie < >>>>> colvin.cowie....@gmail.com> wrote: >>>>> >>>>>> Hello, >>>>>> Eric was going to have a look at the PR. >>>>>> But if it isn't done in time then I don't think it needs to block the >>>>>> release >>>>>> >>>>>> Thanks >>>>>> >>>>>> On Tue, 25 May 2021 at 15:50, Mayya Sharipova < >>>>>> mayya.sharip...@elastic.co.invalid> wrote: >>>>>> >>>>>>> Hello Colvin, >>>>>>> I am wondering if you still want to merge SOLR-15410 for the >>>>>>> Lucene/Solr 8.9 release? >>>>>>> Should we have a deadline for feature freeze? Say May 30th (Sunday)? >>>>>>> >>>>>>> Thank you. >>>>>>> >>>>>>> On Tue, May 18, 2021 at 8:49 AM Noble Paul <noble.p...@gmail.com> >>>>>>> wrote: >>>>>>> >>>>>>>> +1 >>>>>>>> >>>>>>>> >>>>>>>> On Tue, May 18, 2021 at 9:30 PM Colvin Cowie < >>>>>>>> colvin.cowie....@gmail.com> wrote: >>>>>>>> > >>>>>>>> > Hello, >>>>>>>> > >>>>>>>> > I raised SOLR-15410 yesterday with a PR to fix an issue with GC >>>>>>>> logging when using new versions of OpenJ9. It's small, so if somebody >>>>>>>> could >>>>>>>> have a look at it in time for 8.9 that would be great >>>>>>>> > >>>>>>>> > Thanks, >>>>>>>> > Colvin >>>>>>>> > >>>>>>>> > On Thu, 13 May 2021 at 17:52, Nhat Nguyen >>>>>>>> > <nhat.ngu...@elastic.co.invalid> >>>>>>>> wrote: >>>>>>>> >> >>>>>>>> >> Hi Mayya, >>>>>>>> >> >>>>>>>> >> I would like to backport LUCENE-9935, which enables bulk-merge >>>>>>>> for stored fields with index sort, to 8.x this weekend. The patch is >>>>>>>> ready, >>>>>>>> but we prefer to give CI some cycles before backporting. Please let me >>>>>>>> know >>>>>>>> if it's okay with the release plan. >>>>>>>> >> >>>>>>>> >> Thanks, >>>>>>>> >> Nhat >>>>>>>> >> >>>>>>>> >> On Thu, May 13, 2021 at 12:44 PM Gus Heck <gus.h...@gmail.com> >>>>>>>> wrote: >>>>>>>> >>> >>>>>>>> >>> Perhaps https://issues.apache.org/jira/browse/SOLR-15378 >>>>>>>> should be investigated before 8.9, maybe make it a blocker? >>>>>>>> >>> >>>>>>>> >>> On Thu, May 13, 2021 at 1:35 AM Robert Muir <rcm...@gmail.com> >>>>>>>> wrote: >>>>>>>> >>>> >>>>>>>> >>>> Mayya, I created backport for Adrien's issue here, to try to >>>>>>>> help out: >>>>>>>> >>>> https://github.com/apache/lucene-solr/pull/2495 >>>>>>>> >>>> >>>>>>>> >>>> Personally, I felt that merging non-trivial changes from main >>>>>>>> branch >>>>>>>> >>>> to 8.x has some additional risks when cherry-picking: >>>>>>>> >>>> * structural changes in main branch making merging more >>>>>>>> difficult >>>>>>>> >>>> (e.g. LUCENE-9705 reorganization of codec versioning, great >>>>>>>> change >>>>>>>> >>>> moving forwards though) >>>>>>>> >>>> * there are many style changes due to spotless in main branch >>>>>>>> which >>>>>>>> >>>> add noise to merging against old code. >>>>>>>> >>>> * In the specific case of LUCENE-9827, the usual additional >>>>>>>> tricky >>>>>>>> >>>> backwards compatibility for 8.x must be added in the backport >>>>>>>> (due to >>>>>>>> >>>> minor version bumps there) which can go wrong. >>>>>>>> >>>> >>>>>>>> >>>> I still think that particular change is worth considering for >>>>>>>> 8.9, it >>>>>>>> >>>> isn't just a performance bug but also a huge improvement to >>>>>>>> test >>>>>>>> >>>> coverage that helps combat risks. >>>>>>>> >>>> >>>>>>>> >>>> But we should still take some precautions when releasing an >>>>>>>> 8.x IMO: >>>>>>>> >>>> * be mindful of what we are backporting and the risks >>>>>>>> involved: it is harder. >>>>>>>> >>>> * try to let jenkins bake changes in 8.x branches for longer >>>>>>>> than >>>>>>>> >>>> usual? even a few days really helps. >>>>>>>> >>>> >>>>>>>> >>>> On Tue, May 11, 2021 at 1:29 PM Mayya Sharipova >>>>>>>> >>>> <mayya.sharip...@elastic.co.invalid> wrote: >>>>>>>> >>>> > >>>>>>>> >>>> > Thanks everyone, >>>>>>>> >>>> > >>>>>>>> >>>> > Adrien, I am happy to try to be a release manager for this >>>>>>>> release. >>>>>>>> >>>> > >>>>>>>> >>>> > Adrien, and Gus, please let me know when your changes are >>>>>>>> merged to 8.x >>>>>>>> >>>> > >>>>>>>> >>>> > >>>>>>>> >>>> > >>>>>>>> >>>> > On Tue, May 11, 2021 at 10:38 AM Gus Heck < >>>>>>>> gus.h...@gmail.com> wrote: >>>>>>>> >>>> >> >>>>>>>> >>>> >> I'm also looking to find time to get >>>>>>>> https://issues.apache.org/jira/browse/SOLR-14597 into some sort of >>>>>>>> 8x. I've recently completed the back port of 2/3 of the lucene tickets >>>>>>>> that >>>>>>>> are related, and hope to work on the third tomorrow.... >>>>>>>> >>>> >> >>>>>>>> >>>> >> I had some feedback there, but I think folks were waiting >>>>>>>> for the version integrated with the final form of the Lucene tickets >>>>>>>> before >>>>>>>> delving further. Hopefully this week I can start on a patch that does >>>>>>>> that. >>>>>>>> >>>> >> >>>>>>>> >>>> >> On Tue, May 11, 2021 at 10:25 AM Adrien Grand < >>>>>>>> jpou...@gmail.com> wrote: >>>>>>>> >>>> >>> >>>>>>>> >>>> >>> I would like to backport LUCENE-9827 before we release >>>>>>>> 8.9, a performance regression to stored fields merges. I'll work on >>>>>>>> this as >>>>>>>> soon as possible. >>>>>>>> >>>> >>> >>>>>>>> >>>> >>> On Thu, May 6, 2021 at 10:28 PM Adrien Grand < >>>>>>>> jpou...@gmail.com> wrote: >>>>>>>> >>>> >>>> >>>>>>>> >>>> >>>> +1 >>>>>>>> >>>> >>>> >>>>>>>> >>>> >>>> Mayya, are you volunteering to be the release manager? >>>>>>>> >>>> >>>> >>>>>>>> >>>> >>>> Le jeu. 6 mai 2021 à 18:06, Ishan Chattopadhyaya < >>>>>>>> ichattopadhy...@gmail.com> a écrit : >>>>>>>> >>>> >>>>> >>>>>>>> >>>> >>>>> +1 >>>>>>>> >>>> >>>>> >>>>>>>> >>>> >>>>> On Thu, May 6, 2021 at 7:50 PM Mayya Sharipova < >>>>>>>> mayya.sharip...@elastic.co.invalid> wrote: >>>>>>>> >>>> >>>>>> >>>>>>>> >>>> >>>>>> Hello everyone, >>>>>>>> >>>> >>>>>> I was wondering if we can have a 8.9.0 release. It has >>>>>>>> been more than 3 months since 8.8.0 was released. >>>>>>>> >>>> >>>>>> 8.9.0 doesn't need to be the last release in the 8.x >>>>>>>> series. >>>>>>>> >>>> >>>>>> >>>>>>>> >>>> >>>>>> Thanks. >>>>>>>> >>>> >>> >>>>>>>> >>>> >>> >>>>>>>> >>>> >>> >>>>>>>> >>>> >>> -- >>>>>>>> >>>> >>> Adrien >>>>>>>> >>>> >> >>>>>>>> >>>> >> >>>>>>>> >>>> >> >>>>>>>> >>>> >> -- >>>>>>>> >>>> >> http://www.needhamsoftware.com (work) >>>>>>>> >>>> >> http://www.the111shift.com (play) >>>>>>>> >>>> >>>>>>>> >>>> >>>>>>>> --------------------------------------------------------------------- >>>>>>>> >>>> To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org >>>>>>>> >>>> For additional commands, e-mail: dev-h...@lucene.apache.org >>>>>>>> >>>> >>>>>>>> >>> >>>>>>>> >>> >>>>>>>> >>> -- >>>>>>>> >>> http://www.needhamsoftware.com (work) >>>>>>>> >>> http://www.the111shift.com (play) >>>>>>>> >>>>>>>> >>>>>>>> >>>>>>>> -- >>>>>>>> ----------------------------------------------------- >>>>>>>> Noble Paul >>>>>>>> >>>>>>>> >>>>>>>> --------------------------------------------------------------------- >>>>>>>> To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org >>>>>>>> For additional commands, e-mail: dev-h...@lucene.apache.org >>>>>>>> >>>>>>>> >> > >
