bq. because it used to fail sometimes on slow machines It will very likely fail then on those virtboxes, etc.
D. On Thu, Mar 15, 2018 at 5:02 AM, Shalin Shekhar Mangar <[email protected]> wrote: > Hi Erick, > > The test was disabled because it used to fail sometimes on slow machines. I > haven't beasted it to see if that still happens but it probably does. I only > fixed it so that it doesn't always fail. So let's see how Jenkins is doing > and then decide. > > On Thu, Mar 15, 2018 at 3:31 AM, Erick Erickson <[email protected]> > wrote: >> >> Shalin: >> >> Should we remove (actually, comment out with a date?) the BadApple >> annotation for doTestIndexFetchOnMasterRestart? And do you think your >> fixes have any influence on the other BadApple >> (doTestIndexAndConfigReplication)? >> >> There's no problem with un-BadApple-ing test that have been or are >> being worked on, and we'd get more test coverage that way. >> >> Or I can do that on Saturday if you'd prefer, assuming the Jenkins >> BadApple tests don't show failures. >> >> >> >> On Wed, Mar 14, 2018 at 1:57 PM, Shalin Shekhar Mangar >> <[email protected]> wrote: >> > 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. >> >> --------------------------------------------------------------------- >> To unsubscribe, e-mail: [email protected] >> For additional commands, e-mail: [email protected] >> > > > > -- > Regards, > Shalin Shekhar Mangar. --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
