svn commit: r1794801 - /ofbiz/ofbiz-plugins/trunk/lucene/src/main/java/org/apache/ofbiz/content/search/ProductDocument.java

[shijh]

Author: shijh
Date: Thu May 11 09:35:19 2017
New Revision: 1794801

URL: http://svn.apache.org/viewvc?rev=1794801&view=rev
Log:
Improved: Update Apache Solr/Lucene to release 6.2.1
OFBIZ-8316

Removed several comments of this issue in ProductDocument.java.

Thanks: Jacques for the comments and reviews.

Modified:
    
ofbiz/ofbiz-plugins/trunk/lucene/src/main/java/org/apache/ofbiz/content/search/ProductDocument.java

Modified: 
ofbiz/ofbiz-plugins/trunk/lucene/src/main/java/org/apache/ofbiz/content/search/ProductDocument.java
URL: 
http://svn.apache.org/viewvc/ofbiz/ofbiz-plugins/trunk/lucene/src/main/java/org/apache/ofbiz/content/search/ProductDocument.java?rev=1794801&r1=1794800&r2=1794801&view=diff
==============================================================================
--- 
ofbiz/ofbiz-plugins/trunk/lucene/src/main/java/org/apache/ofbiz/content/search/ProductDocument.java
 (original)
+++ 
ofbiz/ofbiz-plugins/trunk/lucene/src/main/java/org/apache/ofbiz/content/search/ProductDocument.java
 Thu May 11 09:35:19 2017
@@ -86,13 +86,8 @@ public class ProductDocument implements
                 this.addTextFieldByWeight(doc, "brandName", 
product.getString("brandName"), "index.weight.Product.brandName", 0, false, 
"fullText", delegator);
                 this.addTextFieldByWeight(doc, "description", 
product.getString("description"), "index.weight.Product.description", 0, false, 
"fullText", delegator);
                 this.addTextFieldByWeight(doc, "longDescription", 
product.getString("longDescription"), "index.weight.Product.longDescription", 
0, false, "fullText", delegator);
-                // OFBiz-8316, the LongPoint should be reused when updating to 
Solr 6.4.1 or later
-//                doc.add(new StringField("introductionDate", 
String.valueOf(quantizeTimestampToDays(product.getTimestamp("introductionDate"))),
 Field.Store.NO));
                 doc.add(new LongPoint("introductionDate", 
quantizeTimestampToDays(product.getTimestamp("introductionDate"))));
                 nextReIndex = 
this.checkSetNextReIndex(product.getTimestamp("introductionDate"), nextReIndex);
-                // OFBiz-8316, the LongPoint should be reused when updating to 
Solr 6.4.1 or later
-//                doc.add(new LongField("salesDiscontinuationDate", 
quantizeTimestampToDays(product.getTimestamp("salesDiscontinuationDate")), 
Field.Store.NO));
-//                doc.add(new StringField("salesDiscontinuationDate", 
String.valueOf(quantizeTimestampToDays(product.getTimestamp("salesDiscontinuationDate"))),
 Field.Store.NO));
                 doc.add(new LongPoint("salesDiscontinuationDate", 
quantizeTimestampToDays(product.getTimestamp("salesDiscontinuationDate"))));
                 nextReIndex = 
this.checkSetNextReIndex(product.getTimestamp("salesDiscontinuationDate"), 
nextReIndex);
                 doc.add(new StringField("isVariant", product.get("isVariant") 
!= null && product.getBoolean("isVariant") ? "true" : "false", Field.Store.NO));
@@ -250,9 +245,6 @@ public class ProductDocument implements
                     fieldNameSb.append('_');
                     
fieldNameSb.append(productPrice.getString("productStoreGroupId"));
                     fieldNameSb.append("_price");
-                    // OFBiz-8316, the DoublePoint should be reused when 
updating to Solr 6.4.1 or later
-//                    doc.add(new DoubleField(fieldNameSb.toString(), 
productPrice.getDouble("price"), Field.Store.NO));
-//                    doc.add(new StringField(fieldNameSb.toString(), 
String.valueOf(productPrice.getDouble("price")), Field.Store.NO));
                     doc.add(new DoublePoint(fieldNameSb.toString(), 
productPrice.getDouble("price")));
                 }