lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Mark Miller <>
Subject Re: being a good citizen is hard when you can't successfully run tests....
Date Mon, 17 Sep 2012 14:49:37 GMT
On Sep 17, 2012, at 6:58 AM, Robert Muir <> wrote:

> On Mon, Sep 17, 2012 at 9:54 AM, Mark Miller <> wrote:
>> It's not so simple. If the replication test fails with the common fail I tend to
see of somehow a searcher not getting closed, I know that's not a big deal for that test.
>> If it failed on something else, I know there might actually be a problem.
> OK, well thats why i'm suggesting in such a case, where:
> 1. this fail isn't considered very important
> and
> 2. its failing every day or multiple times a day
> and
> 3. nobody is planning on fixing it anytime soon
> that we disable the assert so that when a real fail happens, we know
> its a problem.
> See my commit:
> I feel like we can do these targeted assume()'s or whatever (i limited
> this to TestReplicationHandler only on freebsd initially, that seems
> to be where it happens the most), and thats very simple. then our
> build isn't crying wolf.

And I'm not arguing against that, or other creative solutions - I'm arguing against simply
disabling the test.

- Mark
To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message