On Tue, Jul 3, 2012 at 10:40 PM, Chris Hostetter <[email protected]> wrote: > > Yonik: would make sense to at least trigger this in the nightly?... > > final int deleteByQueryPercent = TEST_NIGHTLY ? (1+random().nextInt(7)) : 0;
If the intention is to reduce time required for the test ... I would rather reduce iterations / stop time / etc, than reduce test coverage like this (by disabling testing delete-by-query). But maybe the intention was to temporarily avoid failures because there were known bugs in distributed delete-by-query that yonik is working on (SOLR-3559?), and he plans to re-enable once they are fixed? Or maybe some other unexpected reason? In any event when reducing test coverage suddenly like this, Yonik, please do a more thorough job explaining why. Mike McCandless http://blog.mikemccandless.com --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
