lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Mark Miller <>
Subject Re: Gentle Prod: LUCENE-1626 (flexible getPositionIncrementGap)
Date Mon, 17 Aug 2009 23:31:01 GMT
Hmm ... I don't think it should become protected - is there a good 
reason for this? - it should probably remain public to preserve back compat.

- Mark

Paul Cowan wrote:
> Hi all,
> Just wondering if anyone out there would be interested in taking a 
> quick peek at -- 
> this lets the position increment gap in an analyzer depend on the 
> current term position, which lets you do some cool things with 
> 'one-to-many' relationships (see 
> for the background 
> and explanation).
> It's a very low-impact patch: all existing Analyzer implementations 
> should work fine, as the default implementation of the new method just 
> delegates to the old one.
> The only breaking change is that the original 
> getPositionIncrementGap(String) becomes protected, as now it's merely 
> the default implementation. I think this is unlikely to be a big 
> impact though: the only place this is called inside Lucene outside 
> another analyzer class is in DocInverterPerField, and it seems 
> unlikely to me that any user code would be calling this method 
> directly (happy to be corrected here).
> I'm asking for purely selfish reasons, if 2.9 is getting very close 
> then if this makes it in, it's one less change to 2.9 we have to 
> maintain in our own repo. Not saying it SHOULD get in, but hey, worth 
> a crack.
> That patch is slightly old (done against 2.4, I think), but is very 
> similar to what's required against trunk. If people seem amenable to 
> landing this (either pre- or post-2.9) then I'll be sure produce a new 
> patch which applies cleanly.
> Let me know what you think!
> Cheers,
> Paul
> ---------------------------------------------------------------------
> To unsubscribe, e-mail:
> For additional commands, e-mail:

- Mark

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

View raw message