You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by ry...@apache.org on 2015/04/10 01:44:44 UTC

svn commit: r1672518 - in /lucene/dev/branches/branch_5x/solr/core/src: java/org/apache/solr/schema/BBoxField.java test/org/apache/solr/search/TestSolr4Spatial.java

Author: ryan
Date: Thu Apr  9 23:44:43 2015
New Revision: 1672518

URL: http://svn.apache.org/r1672518
Log:
SOLR-7164: update lucene field also

Modified:
    lucene/dev/branches/branch_5x/solr/core/src/java/org/apache/solr/schema/BBoxField.java
    lucene/dev/branches/branch_5x/solr/core/src/test/org/apache/solr/search/TestSolr4Spatial.java

Modified: lucene/dev/branches/branch_5x/solr/core/src/java/org/apache/solr/schema/BBoxField.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_5x/solr/core/src/java/org/apache/solr/schema/BBoxField.java?rev=1672518&r1=1672517&r2=1672518&view=diff
==============================================================================
--- lucene/dev/branches/branch_5x/solr/core/src/java/org/apache/solr/schema/BBoxField.java (original)
+++ lucene/dev/branches/branch_5x/solr/core/src/java/org/apache/solr/schema/BBoxField.java Thu Apr  9 23:44:43 2015
@@ -139,6 +139,8 @@ public class BBoxField extends AbstractS
     final SchemaField solrNumField = new SchemaField("_", numberType);//dummy temp
     org.apache.lucene.document.FieldType luceneType =
         (org.apache.lucene.document.FieldType) solrNumField.createField(0.0, 1.0f).fieldType();
+    luceneType.setStored(storeSubFields);
+    
     //and annoyingly this Field isn't going to have a docValues format because Solr uses a separate Field for that
     if (solrNumField.hasDocValues()) {
       luceneType = new org.apache.lucene.document.FieldType(luceneType);

Modified: lucene/dev/branches/branch_5x/solr/core/src/test/org/apache/solr/search/TestSolr4Spatial.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_5x/solr/core/src/test/org/apache/solr/search/TestSolr4Spatial.java?rev=1672518&r1=1672517&r2=1672518&view=diff
==============================================================================
--- lucene/dev/branches/branch_5x/solr/core/src/test/org/apache/solr/search/TestSolr4Spatial.java (original)
+++ lucene/dev/branches/branch_5x/solr/core/src/test/org/apache/solr/search/TestSolr4Spatial.java Thu Apr  9 23:44:43 2015
@@ -382,6 +382,11 @@ public class TestSolr4Spatial extends So
       // Make sure the subfields are not stored
       SchemaField sub = schema.getField("bbox"+BBoxStrategy.SUFFIX_MINX);
       assertFalse(sub.stored());
+
+      // Make sure solr field type is also not stored
+      BBoxField bbox = (BBoxField)schema.getField("bbox").getType();
+      BBoxStrategy strategy = bbox.getStrategy("bbox");
+      assertFalse(strategy.getFieldType().stored());
     }
   }