lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Paul Elschot <>
Subject Re: Tests, Contribs, and Releases
Date Sat, 19 May 2007 08:19:23 GMT
On Friday 18 May 2007 18:52, Doug Cutting wrote:
> Paul Elschot wrote:
> > When a contrib fails and is not fixed, that might be a good reason to
> > remove it from the distribution. With such a policy the present contribs 
> > also stay up to date, provided their tests are good enough.
> > Thinking about it a bit more, there is a process to add contribs, but 
> > is not really a process to remove them. The policy of removing a contrib
> > before a release when its unit tests fail could make a nice fit for that.
> I think what you're arguing is that, unlike the core, contrib is not 
> guaranteed to be back-compatible (since removing a module is not a 
> back-compatible change).  Otherwise this is no different than other bugs 
> and code changes.  If a contrib package is failing tests or breaking the 
> build, then we should file an issue in Jira.  One potential patch is to 
> remove the package (since incompatible changes are permitted in 
> contrib).  That can be proposed, and if no one with a binding vote 
> objects, it can happen.  We shouldn't release with unresolved issues, 
> whether they involve contrib or core.  Does that sound reasonable?

The idea is roughly to give interested people enough time to fix
a contrib when some of its tests fail. In case no fix is made
available, one can reasonably conclude that there is no more
interest, so removing the contrib can be considered.
An impending release is a good moment for that.

Btw. this presupposes that the contrib unit tests are good enough
from the start, and except for contrib/surround I don't know about
Good enough means that such tests implicitly verify all of
the lucene core library that the contrib can be expected to
normally use.
As long as they pass, such contrib tests are also a good motivation
to keep the contrib close to the core.

I had not considered contribs that depend things other than the
lucene core, but discussion on that has already started.

Paul Elschot

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

View raw message