This is an automated email from the ASF dual-hosted git repository.

markus pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/nutch.git


The following commit(s) were added to refs/heads/master by this push:
     new a67c9be  NUTCH-2724 Metadata indexer not to emit empty values
a67c9be is described below

commit a67c9bee94049d37dad9278cdf8dd9131735da43
Author: Markus Jelsma <mar...@apache.org>
AuthorDate: Mon Jul 15 12:25:42 2019 +0200

    NUTCH-2724 Metadata indexer not to emit empty values
---
 .../java/org/apache/nutch/indexer/metadata/MetadataIndexer.java    | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git 
a/src/plugin/index-metadata/src/java/org/apache/nutch/indexer/metadata/MetadataIndexer.java
 
b/src/plugin/index-metadata/src/java/org/apache/nutch/indexer/metadata/MetadataIndexer.java
index 3927bd3..be56377 100644
--- 
a/src/plugin/index-metadata/src/java/org/apache/nutch/indexer/metadata/MetadataIndexer.java
+++ 
b/src/plugin/index-metadata/src/java/org/apache/nutch/indexer/metadata/MetadataIndexer.java
@@ -93,12 +93,15 @@ public class MetadataIndexer implements IndexingFilter {
   
   protected void add(NutchDocument doc, String key, String value) {
     if (separator == null || value.indexOf(separator) == -1 || 
!mvFields.contains(key)) {
-      doc.add(key, value);
+      value = value.trim();
+      if (!value.isEmpty()) {
+        doc.add(key, value);
+      }
     } else {
       String[] parts = value.split(separator);
       for (String part : parts) {
         part = part.trim();
-        if (part.length() != 0) {
+        if (!part.isEmpty()) {
           doc.add(key, part);
         }
       }

Reply via email to