Author: ab
Date: Thu Jan 26 15:48:14 2006
New Revision: 372664

URL: http://svn.apache.org/viewcvs?rev=372664&view=rev
Log:
Fix for NUTCH-190 (ParseUtil drops reason for failed parse).

Also, hide the message about successful parse behind LOG.fine() -
no message is a good message; if users want this they can still have it.


Modified:
    lucene/nutch/trunk/src/java/org/apache/nutch/parse/ParseUtil.java

Modified: lucene/nutch/trunk/src/java/org/apache/nutch/parse/ParseUtil.java
URL: 
http://svn.apache.org/viewcvs/lucene/nutch/trunk/src/java/org/apache/nutch/parse/ParseUtil.java?rev=372664&r1=372663&r2=372664&view=diff
==============================================================================
--- lucene/nutch/trunk/src/java/org/apache/nutch/parse/ParseUtil.java (original)
+++ lucene/nutch/trunk/src/java/org/apache/nutch/parse/ParseUtil.java Thu Jan 
26 15:48:14 2006
@@ -64,7 +64,7 @@
     
     Parse parse = null;
     for (int i=0; i<parsers.length; i++) {
-      LOG.info("Parsing [" + content.getUrl() + "] with [" + parsers[i] + "]");
+      LOG.fine("Parsing [" + content.getUrl() + "] with [" + parsers[i] + "]");
       parse = parsers[i].getParse(content);
       if ((parse != null) && (parse.getData().getStatus().isSuccess())) {
         return parse;
@@ -74,7 +74,8 @@
     LOG.warning("Unable to successfully parse content " + content.getUrl() +
                 " of type " + content.getContentType());
 
-    return new ParseStatus().getEmptyParse();
+    ParseStatus ps = (parse.getData() != null) ? parse.getData().getStatus() : 
null;
+    return (ps == null) ? new ParseStatus().getEmptyParse() : 
ps.getEmptyParse();
   }
   
   /**


Reply via email to