You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by sh...@apache.org on 2013/05/16 15:17:54 UTC
svn commit: r1483347 - in /lucene/dev/branches/lucene_solr_4_3: ./ lucene/
lucene/CHANGES.txt lucene/core/
lucene/core/src/java/org/apache/lucene/search/NGramPhraseQuery.java
lucene/core/src/test/org/apache/lucene/search/TestNGramPhraseQuery.java
Author: shalin
Date: Thu May 16 13:17:54 2013
New Revision: 1483347
URL: http://svn.apache.org/r1483347
Log:
LUCENE-4970: Fix boost value of rewritten NGramPhraseQuery.
Modified:
lucene/dev/branches/lucene_solr_4_3/ (props changed)
lucene/dev/branches/lucene_solr_4_3/lucene/ (props changed)
lucene/dev/branches/lucene_solr_4_3/lucene/CHANGES.txt (contents, props changed)
lucene/dev/branches/lucene_solr_4_3/lucene/core/ (props changed)
lucene/dev/branches/lucene_solr_4_3/lucene/core/src/java/org/apache/lucene/search/NGramPhraseQuery.java
lucene/dev/branches/lucene_solr_4_3/lucene/core/src/test/org/apache/lucene/search/TestNGramPhraseQuery.java
Modified: lucene/dev/branches/lucene_solr_4_3/lucene/CHANGES.txt
URL: http://svn.apache.org/viewvc/lucene/dev/branches/lucene_solr_4_3/lucene/CHANGES.txt?rev=1483347&r1=1483346&r2=1483347&view=diff
==============================================================================
--- lucene/dev/branches/lucene_solr_4_3/lucene/CHANGES.txt (original)
+++ lucene/dev/branches/lucene_solr_4_3/lucene/CHANGES.txt Thu May 16 13:17:54 2013
@@ -29,6 +29,9 @@ Bug Fixes
the child query has no hits, more aggressively catch cases where childQuery
incorrectly matches parent documents (Mike McCandless)
+* LUCENE-4970: Fix boost value of rewritten NGramPhraseQuery.
+ (Shingo Sasaki via Adrien Grand)
+
======================= Lucene 4.3.0 =======================
Changes in backwards compatibility policy
Modified: lucene/dev/branches/lucene_solr_4_3/lucene/core/src/java/org/apache/lucene/search/NGramPhraseQuery.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/lucene_solr_4_3/lucene/core/src/java/org/apache/lucene/search/NGramPhraseQuery.java?rev=1483347&r1=1483346&r2=1483347&view=diff
==============================================================================
--- lucene/dev/branches/lucene_solr_4_3/lucene/core/src/java/org/apache/lucene/search/NGramPhraseQuery.java (original)
+++ lucene/dev/branches/lucene_solr_4_3/lucene/core/src/java/org/apache/lucene/search/NGramPhraseQuery.java Thu May 16 13:17:54 2013
@@ -64,6 +64,7 @@ public class NGramPhraseQuery extends Ph
// now create the new optimized phrase query for n-gram
PhraseQuery optimized = new PhraseQuery();
+ optimized.setBoost(getBoost());
int pos = 0;
final int lastPos = terms.length - 1;
for(int i = 0; i < terms.length; i++){
Modified: lucene/dev/branches/lucene_solr_4_3/lucene/core/src/test/org/apache/lucene/search/TestNGramPhraseQuery.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/lucene_solr_4_3/lucene/core/src/test/org/apache/lucene/search/TestNGramPhraseQuery.java?rev=1483347&r1=1483346&r2=1483347&view=diff
==============================================================================
--- lucene/dev/branches/lucene_solr_4_3/lucene/core/src/test/org/apache/lucene/search/TestNGramPhraseQuery.java (original)
+++ lucene/dev/branches/lucene_solr_4_3/lucene/core/src/test/org/apache/lucene/search/TestNGramPhraseQuery.java Thu May 16 13:17:54 2013
@@ -88,6 +88,17 @@ public class TestNGramPhraseQuery extend
pq3 = (PhraseQuery)q;
assertArrayEquals(new Term[]{new Term("f", "ABC"), new Term("f", "DEF"), new Term("f", "FGH")}, pq3.getTerms());
assertArrayEquals(new int[]{0, 3, 5}, pq3.getPositions());
+
+ // LUCENE-4970: boosting test
+ PhraseQuery pq4 = new NGramPhraseQuery(2);
+ pq4.add(new Term("f", "AB"));
+ pq4.add(new Term("f", "BC"));
+ pq4.add(new Term("f", "CD"));
+ pq4.setBoost(100.0F);
+
+ q = pq4.rewrite(reader);
+ assertNotSame(pq4, q);
+ assertEquals(pq4.getBoost(), q.getBoost(), 0.1f);
}
}