[mb-commits] r13649 - in search_server/trunk/index/src/main/java/org/musicbrainz/search: analysis index

[email protected] Thu, 14 Mar 2013 10:26:26 +0000
Newsgroups gmane.comp.audio.musicbrainz.cvs
Message-ID <E1UG5Mg-0001Kh-DU@wiley>
Author: ijabz
Date: 2013-03-14 10:26:25 +0000 (Thu, 14 Mar 2013)
New Revision: 13649

Modified:
   search_server/trunk/index/src/main/java/org/musicbrainz/search/analysis/RecordingSimilarity.java
   search_server/trunk/index/src/main/java/org/musicbrainz/search/index/RecordingIndex.java
Log:
Reinstated missing Recording SImailrity class

Modified: search_server/trunk/index/src/main/java/org/musicbrainz/search/analysis/RecordingSimilarity.java
===================================================================
--- search_server/trunk/index/src/main/java/org/musicbrainz/search/analysis/RecordingSimilarity.java	2013-02-28 13:30:23 UTC (rev 13648)
+++ search_server/trunk/index/src/main/java/org/musicbrainz/search/analysis/RecordingSimilarity.java	2013-03-14 10:26:25 UTC (rev 13649)
@@ -37,6 +37,8 @@
 /**
  * Calculates a score for a match, overridden to deal with problems with recordings linked to many releases
  */
+//TODO in Lucene 4.1 we can now use PerFieldSimailrityWrapper so that we only oerform this on fields that need it, with
+//current code tf() is performed on every field because we are not passed fieldname
 public class RecordingSimilarity extends DefaultSimilarity {
 
     /**

Modified: search_server/trunk/index/src/main/java/org/musicbrainz/search/index/RecordingIndex.java
===================================================================
--- search_server/trunk/index/src/main/java/org/musicbrainz/search/index/RecordingIndex.java	2013-02-28 13:30:23 UTC (rev 13648)
+++ search_server/trunk/index/src/main/java/org/musicbrainz/search/index/RecordingIndex.java	2013-03-14 10:26:25 UTC (rev 13649)
@@ -26,6 +26,7 @@
 import org.apache.lucene.search.similarities.Similarity;
 import org.musicbrainz.mmd2.*;
 import org.musicbrainz.search.MbDocument;
+import org.musicbrainz.search.analysis.RecordingSimilarity;
 
 import java.io.IOException;
 import java.math.BigInteger;
@@ -100,12 +101,10 @@
     String releases;
     String releaseSecondaryTypes;
 
-    /*
-    FISME
     @Override
     public Similarity getSimilarity() {
         return new RecordingSimilarity();
-    } */
+    }
 
     public void init(IndexWriter indexWriter, boolean isUpdater) throws SQLException {