You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nutch.apache.org by ma...@apache.org on 2012/01/27 14:11:47 UTC

svn commit: r1236674 - in /nutch/trunk: CHANGES.txt src/java/org/apache/nutch/fetcher/Fetcher.java

Author: markus
Date: Fri Jan 27 13:11:47 2012
New Revision: 1236674

URL: http://svn.apache.org/viewvc?rev=1236674&view=rev
Log:
NUTCH-1260 Fetcher should log fetching of redirects

Modified:
    nutch/trunk/CHANGES.txt
    nutch/trunk/src/java/org/apache/nutch/fetcher/Fetcher.java

Modified: nutch/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/nutch/trunk/CHANGES.txt?rev=1236674&r1=1236673&r2=1236674&view=diff
==============================================================================
--- nutch/trunk/CHANGES.txt (original)
+++ nutch/trunk/CHANGES.txt Fri Jan 27 13:11:47 2012
@@ -1,5 +1,7 @@
 Nutch Change Log
 
+* NUTCH-1260 Fetcher should log fetching of redirects (Sebastian Nagel via markus)
+
 * NUTCH-1255 Change ivy.xml of all plugins to remove "nutch.root" property (ferdy)
 
 * NUTCH-1248 Generator to select on status (markus)

Modified: nutch/trunk/src/java/org/apache/nutch/fetcher/Fetcher.java
URL: http://svn.apache.org/viewvc/nutch/trunk/src/java/org/apache/nutch/fetcher/Fetcher.java?rev=1236674&r1=1236673&r2=1236674&view=diff
==============================================================================
--- nutch/trunk/src/java/org/apache/nutch/fetcher/Fetcher.java (original)
+++ nutch/trunk/src/java/org/apache/nutch/fetcher/Fetcher.java Fri Jan 27 13:11:47 2012
@@ -653,12 +653,11 @@ public class Fetcher extends Configured 
             reprUrl = reprUrlWritable.toString();
           }
           try {
-            if (LOG.isInfoEnabled()) { LOG.info("fetching " + fit.url); }
-
             // fetch the page
             redirecting = false;
             redirectCount = 0;
             do {
+              if (LOG.isInfoEnabled()) { LOG.info("fetching " + fit.url); }
               if (LOG.isDebugEnabled()) {
                 LOG.debug("redirectCount=" + redirectCount);
               }