Robert Chapman ported Spatial to Lucene.NET, shown here in this issue: https://issues.apache.org/jira/browse/LUCENENET-199. DIGY only committed the patch, and the only thing I've seen Prescott do was to modify/apply LUCENENET-431 to trunk from 2.9.4g. Spatial, along with the rest of Contrib, was barely touched since Lucene.NET was resurrected into the incubator. Most, if not all, of the code is the exact same as the patch contributed by Robert, which itself differed a decent amount from the java code, so much of the reasons why it wasn't an exact port are either lost, or the more likely case, never answered.
As far as I'm concerned, I have no qualms with you rewriting the thing from scratch, if you wanted to pursue a more direct port from the java code. Aside from the original port which had many divergences from java, the only other issue applied to spatial is LUCENENET-431, which would be easy to include. Thanks, Christopher On Tue, Apr 24, 2012 at 10:31 AM, Itamar Syn-Hershko <ita...@code972.com>wrote: > Uhm.. I was referring to the .NET port, which I can see DIGY ported > > Nevermind I will get it from the original commit > > @Prescott any idea re " CartesianPolyFilterBuilder.GetBoxShape() is not an > exact port - do you remember why? "? > > On Tue, Apr 24, 2012 at 12:26 AM, Christopher Currens < > currens.ch...@gmail.com> wrote: > > > It's in a weird place. And for the 3.0.3 version, its easiest to find > the > > code in the tags, rather than branches. > > > > > > > http://svn.apache.org/viewvc/lucene/java/tags/lucene_3_0_3/contrib/misc/src/java/org/apache/lucene/misc/ > > > > > > On Mon, Apr 23, 2012 at 2:20 PM, Prescott Nasser <geobmx...@hotmail.com > > >wrote: > > > > > > > > I'm having trouble finding chained filter in the java lucene svn... > > > > > > http://svn.apache.org/viewvc/lucene/dev/branches/branch_3x/lucene/contrib/?pathrev=990167amIlooking > around in the wrong place? > > > > Date: Mon, 23 Apr 2012 11:19:51 +0300 > > > > Subject: Re: Spatial contrib bug fixing > > > > From: ita...@code972.com > > > > To: lucene-net-...@lucene.apache.org > > > > > > > > One more thing - what's the deal with ChainedFilter? I can see a > commit > > > by > > > > DIGY on 7/7/2011 but it seems to have been removed since? > > > > > > > > On Mon, Apr 23, 2012 at 11:06 AM, Itamar Syn-Hershko < > > ita...@code972.com > > > >wrote: > > > > > > > > > For starters - CartesianPolyFilterBuilder.GetBoxShape() is not an > > exact > > > > > port - do you remember why? > > > > > > > > > > Anyway, if it was never fully ported as you say maybe I'll just go > > > ahead > > > > > and complete that > > > > > > > > > > For your reference, here are 2 failing tests which pass in Java > > Lucene > > > > > (can send the java file) - > > > > > > > > > > > https://github.com/synhershko/lucene.net/commit/234da7eca7cb08be5a0c2a7375ffc3f4a03bfd92 > > > > > > > > > > > > > > > > > > > > On Mon, Apr 23, 2012 at 1:39 AM, Prescott Nasser < > > > geobmx...@hotmail.com>wrote: > > > > > > > > > >> > > > > >> I think that was a while ago, and I don't even recall if I fully > > > ported > > > > >> it or just put up the start. I had some other stuff to deal with > the > > > last > > > > >> few months, so my memory is a bit lacking. I'll review the code, > > > meanwhile > > > > >> ask whatever questions you have - lets get this fixed up. ~P > > > > >> > Date: Sun, 22 Apr 2012 22:10:27 +0300 > > > > >> > Subject: Spatial contrib bug fixing > > > > >> > From: ita...@code972.com > > > > >> > To: lucene-net-...@lucene.apache.org > > > > >> > > > > > >> > Hi all, > > > > >> > > > > > >> > We encountered several bugs with the Sparial contrb, and the > ones > > we > > > > >> tested > > > > >> > with Java Lucene worked there (with 2.9.4). There are about 3 > open > > > > >> tickets > > > > >> > in the Jira bug tracker on similar issues. > > > > >> > > > > > >> > I'm now sitting with the ultimate goal of fixing this once and > for > > > all, > > > > >> but > > > > >> > some code parts are commented out in favor of other not > > line-by-line > > > > >> port > > > > >> > of some implementations, without a comment giving reasons. I was > > > > >> wondering > > > > >> > if there's anyone who could answer a few questions there, > instead > > > of me > > > > >> > changing things back and forth? > > > > >> > > > > > >> > Git history (I use the Git mirror, yes) tells me Prescott Nasser > > is > > > > >> behind > > > > >> > porting this - maybe he will have the answers? > > > > >> > > > > > >> > Cheers, > > > > >> > > > > > >> > Itamar. > > > > >> > > > > >> > > > > > > > > > > > > > > > > > > >
