This is fixed. I committed the fix to master, branch_7x and branch_7_3 branches.
On Wed, Mar 14, 2018 at 9:44 PM, Alan Woodward <[email protected]> wrote: > Thanks Shalin! > > > On 14 Mar 2018, at 15:50, Shalin Shekhar Mangar <[email protected]> > wrote: > > I'll take a look at it tomorrow morning my time. > > On Wed, Mar 14, 2018 at 9:07 PM, Andrzej Białecki < > [email protected]> wrote: > >> Well … I looked at it briefly but I have no idea what’s going on there. I >> could dig into it nonetheless, but if there’s someone who already knows the >> replication handler ins and outs it would probably get fixed sooner... >> >> >> On 14 Mar 2018, at 14:23, Alan Woodward <[email protected]> wrote: >> >> I’m happy either way, but if it’s a bug can we get it fixed quickly? Can >> you take ownership of this one Andrzej? >> >> On 14 Mar 2018, at 11:24, Andrzej Białecki <[email protected]> wrote: >> >> Hi, >> >> This test has always been fragile, but recently it’s been failing 100%, >> most often in ‘doTestIndexFetchOnMasterRestart’. >> >> I don’t know the replication handler enough to be able to find the real >> reason behind these failures, but there are two possibilities that I see: >> >> * the test has a bug and needs to be fixed - and if we can’t fix it soon >> then with 7.3 release imminent we could BadApple it until it’s properly >> fixed >> >> * or actually the replication handler has a bug, which needs to be fixed >> - in which case I propose to bump up SOLR-12078 to Blocker. >> >> I’m open to suggestions. >> >> — >> >> Andrzej Białecki >> >> >> >> > > > -- > Regards, > Shalin Shekhar Mangar. > > > -- Regards, Shalin Shekhar Mangar.
