Thanks so much for getting the back compat stuff started Chris, that's a lot of help.
I'm comparing these to the commits I made for the 8.11.3 release, and there are some discrepancies. But we can take care of that on the PRs. - Houston On Wed, Sep 25, 2024 at 8:37 AM Chris Hegarty < christopher.hega...@elastic.co> wrote: > Hi, > > > On 25 Sep 2024, at 13:08, Michael McCandless <luc...@mikemccandless.com> > wrote: > > > > On Wed, Sep 25, 2024 at 7:01 AM Chris Hegarty < > christopher.hega...@elastic.co> wrote: > > > > > +1 for release, unless the uncovered issue with TieredMergePolicy > causes malfunction. > > > > My analysis concludes that there is no product bug here. Yeah, maybe the > policy could be a little more aggressive in its merging, but the issue that > the test runs into looks like a corner case on the boundary of what the > heuristics look for. > > > > https://github.com/apache/lucene/issues/13818#issuecomment-2371868712 > > > > It would be good to have Adrien or Mike confirm. > > > > I added a comment -- this looks like a rare corner case bug or test > issue. It should not be a 9.12.0 release blocker > > Thank you Mike. > > Ok, so no blocker bugs found. We have some votes, not too many, but maybe > enough. > > Last thing. Backward compat with the recently released 8.11.4. I finally > managed to build the lucene-solr and generate these bw compat indices. I > opened the following PR’s - one for main and one for branch_9_12: > > * https://github.com/apache/lucene/pull/13824 > * https://github.com/apache/lucene/pull/13823 > > We don’t strictly need it, but the smoke testing may fail. If I can get > some sanity on the PR’s, and they look sufficient, then I’m happy to respin > a new RC for 9.12 - to include support for 8.11.4 ( in the amount of the > version constant and test verification ) > > -Chris.
