- changed handling of error cases retrieving urls from database

(no more NULL values are returned, instead, an IOException is thrown)
- removed ugly damagedURLS implementation from plasmaCrawlLURL.java
  (this inserted a static value into the Object which is not really a good style)
- re-coded damagedURLS collection in yacy.java by catching an exception and evaluating the exception message
to do:
- the urldbcleanup feature must be re-tested


git-svn-id: https://svn.berlios.de/svnroot/repos/yacy/trunk@1200 6c8d7289-2bf4-0310-a012-ef5d649a1542

htroot/IndexControl_p.java

@@ -47,7 +47,6 @@
 // if the shell's current path is HTROOT
 
 import java.io.IOException;
-import java.net.MalformedURLException;
 import java.net.URL;
 import java.util.Enumeration;
 import java.util.HashSet;
@@ -167,9 +166,9 @@ public class IndexControl_p {
                 }
             }
             if (delurlref) {
-                for (int i = 0; i < urlx.length; i++) {
+                for (int i = 0; i < urlx.length; i++) try {
                     switchboard.removeAllUrlReferences(urlx[i], true);
-                }
+                } catch (IOException e) {}
             }
             if (delurl || delurlref) {
                 for (int i = 0; i < urlx.length; i++) {
@@ -189,9 +188,9 @@ public class IndexControl_p {
         // delete selected URLs
         if (post.containsKey("keyhashdelete")) {
             if (delurlref) {
-                for (int i = 0; i < urlx.length; i++) {
+                for (int i = 0; i < urlx.length; i++) try {
                     switchboard.removeAllUrlReferences(urlx[i], true);
-                }
+                } catch (IOException e) {}
             }
             if (delurl || delurlref) {
                 for (int i = 0; i < urlx.length; i++) {
@@ -212,20 +211,24 @@ public class IndexControl_p {
         }
 
         if (post.containsKey("urlhashdeleteall")) {
+            try {
                 int i = switchboard.removeAllUrlReferences(urlhash, true);
                 prop.put("result", "Deleted URL and " + i + " references from " + i + " word indexes.");
+            } catch (IOException e) {
+                prop.put("result", "Deleted nothing because the url-hash could not be resolved");
+            }
         }
 
         if (post.containsKey("urlhashdelete")) {
+            try {
                 plasmaCrawlLURL.Entry entry = switchboard.urlPool.loadedURL.getEntry(urlhash);
                 URL url = entry.url();
-            if (url == null) {
-                prop.put("result", "No Entry for URL hash " + urlhash + "; nothing deleted.");
-            } else {
                 urlstring = htmlFilterContentScraper.urlNormalform(url);
                 prop.put("urlstring", "");
                 switchboard.urlPool.loadedURL.remove(urlhash);
                 prop.put("result", "Removed URL " + urlstring);
+            } catch (IOException e) {
+                prop.put("result", "No Entry for URL hash " + urlhash + "; nothing deleted.");
             }
         }
 
@@ -267,16 +270,16 @@ public class IndexControl_p {
             plasmaCrawlLURL.Entry lurl;
             while (urlIter.hasNext()) {
                 indexEntry = (plasmaWordIndexEntry) urlIter.next();
+                try {
                     lurl = switchboard.urlPool.loadedURL.getEntry(indexEntry.getUrlHash());
-                if (lurl == null) {
-                    unknownURLEntries.add(indexEntry.getUrlHash());
-                } else {
                     if (lurl.toString() == null) {
                         switchboard.urlPool.loadedURL.remove(indexEntry.getUrlHash());
                         unknownURLEntries.add(indexEntry.getUrlHash());
                     } else {
                         knownURLs.put(indexEntry.getUrlHash(), lurl);
                     }
+                } catch (IOException e) {
+                    unknownURLEntries.add(indexEntry.getUrlHash());
                 }
             }
             // now delete all entries that have no url entry
@@ -327,21 +330,21 @@ public class IndexControl_p {
                 prop.put("urlhash", urlhash);
                 plasmaCrawlLURL.Entry entry = switchboard.urlPool.loadedURL.getEntry(urlhash);
                 prop.put("result", genUrlProfile(switchboard, entry, urlhash));
-            } catch (MalformedURLException e) {
+            } catch (Exception e) {
                 prop.put("urlstring", "wrong url: " + urlstring);
                 prop.put("urlhash", "");
             }
         }
 
         if (post.containsKey("urlhashsearch")) {
+            try {
                 plasmaCrawlLURL.Entry entry = switchboard.urlPool.loadedURL.getEntry(urlhash);
                 URL url = entry.url();
-            if (url == null) {
-                prop.put("result", "No Entry for URL hash " + urlhash);
-            } else {
                 urlstring = url.toString();
                 prop.put("urlstring", urlstring);
                 prop.put("result", genUrlProfile(switchboard, entry, urlhash));
+            } catch (IOException e) {
+                prop.put("result", "No Entry for URL hash " + urlhash);
             }
         }
 
@@ -391,6 +394,12 @@ public class IndexControl_p {
     public static String genUrlProfile(plasmaSwitchboard switchboard, plasmaCrawlLURL.Entry entry, String urlhash) {
         if (entry == null) { return "No entry found for URL-hash " + urlhash; }
         URL url = entry.url();
+        String referrer = null;
+        try {
+            referrer = switchboard.urlPool.loadedURL.getEntry(entry.referrerHash()).url().toString();
+        } catch (IOException e) {
+            referrer = "<unknown>";
+        }
         if (url == null) { return "No entry found for URL-hash " + urlhash; }
         String result = "<table>" +
         "<tr><td class=\"small\">URL String</td><td class=\"tt\">" + htmlFilterContentScraper.urlNormalform(url) + "</td></tr>" +
@@ -398,7 +407,7 @@ public class IndexControl_p {
         "<tr><td class=\"small\">Description</td><td class=\"tt\">" + entry.descr() + "</td></tr>" +
         "<tr><td class=\"small\">Modified-Date</td><td class=\"tt\">" + entry.moddate() + "</td></tr>" +
         "<tr><td class=\"small\">Loaded-Date</td><td class=\"tt\">" + entry.loaddate() + "</td></tr>" +
-        "<tr><td class=\"small\">Referrer</td><td class=\"tt\">" + switchboard.urlPool.loadedURL.getEntry(entry.referrerHash()).url() + "</td></tr>" +
+        "<tr><td class=\"small\">Referrer</td><td class=\"tt\">" + referrer + "</td></tr>" +
         "<tr><td class=\"small\">Doctype</td><td class=\"tt\">" + entry.doctype() + "</td></tr>" +
         "<tr><td class=\"small\">Copy-Count</td><td class=\"tt\">" + entry.copyCount() + "</td></tr>" +
         "<tr><td class=\"small\">Local-Flag</td><td class=\"tt\">" + entry.local() + "</td></tr>" +

htroot/IndexCreateWWWLocalQueue_p.java

@@ -43,6 +43,7 @@
 // javac -classpath .:../classes IndexCreate_p.java
 // if the shell's current path is HTROOT
 
+import java.io.IOException;
 import java.text.SimpleDateFormat;
 import java.util.Date;
 import java.util.Iterator;
@@ -96,9 +97,12 @@ public class IndexCreateWWWLocalQueue_p {
                         while (iter.hasNext()) {
                             String value = null;
                             String nextHash = new String((byte[]) iter.next());
-                            Entry entry = switchboard.urlPool.noticeURL.getEntry(nextHash);
-                            if (entry == null) continue;
-                            
+                            Entry entry = null;
+                            try {
+                                entry = switchboard.urlPool.noticeURL.getEntry(nextHash);
+                            } catch (IOException e) {
+                                continue;
+                            }
                             if ((option.equals("URL")&&(entry.url() != null))) {
                                 value = entry.url().toString();
                             } else if ((option.equals("AnchorName"))) {

htroot/ViewFile.java

@@ -102,8 +102,10 @@ public class ViewFile {
             String viewMode = post.get("viewMode","sentences");
             
             // getting the urlEntry that belongs to the url hash
-            Entry urlEntry = sb.urlPool.loadedURL.getEntry(urlHash);
-            if (urlEntry == null) {
+            Entry urlEntry = null;
+            try {
+                urlEntry = sb.urlPool.loadedURL.getEntry(urlHash);
+            } catch (IOException e) {
                 prop.put("error",2);
                 prop.put("viewMode",VIEW_MODE_NO_TEXT);
                 return prop;

htroot/yacy/crawlOrder.java

@@ -45,6 +45,7 @@
 // You must compile this file with
 // javac -classpath .:../classes crawlOrder.java
 
+import java.io.IOException;
 import java.util.ArrayList;
 import java.util.Date;
 import de.anomic.htmlFilter.htmlFilterContentScraper;
@@ -243,12 +244,12 @@ public final class crawlOrder {
             // case where we have already the url loaded;
             reason = reasonString;
             // send lurl-Entry as response
+            try {
                 plasmaCrawlLURL.Entry entry = switchboard.urlPool.loadedURL.getEntry(plasmaURL.urlHash(url));
-            if (entry != null) {
                 response = "double";
                 switchboard.urlPool.loadedURL.notifyGCrawl(entry.hash(), iam, youare);
                 lurl = crypt.simpleEncode(entry.toString());
-            } else {
+            } catch (IOException e) {
                 response = "rejected";
                 lurl = "";
             }

htroot/yacy/crawlReceipt.java

@@ -43,6 +43,8 @@
 // javac -classpath .:../classes crawlOrder.java
 
 
+import java.io.IOException;
+
 import de.anomic.http.httpHeader;
 import de.anomic.plasma.plasmaCrawlNURL;
 import de.anomic.plasma.plasmaCrawlLURL;
@@ -135,10 +137,12 @@ public final class crawlReceipt {
             // ready for more
             prop.put("delay", "10");
         } else {
+            try {
                 plasmaCrawlNURL.Entry en = switchboard.urlPool.noticeURL.getEntry(receivedUrlhash);
-            if (en != null) {
                 switchboard.urlPool.errorURL.newEntry(en.url(), en.referrerHash(), en.initiator(), iam, en.name(), result + ":" + reason, new bitfield(plasmaURL.urlFlagLength), false);
                 switchboard.urlPool.noticeURL.remove(receivedUrlhash);
+            } catch (IOException e) {
+
             }
             prop.put("delay", "100"); // what shall we do with that???
         }

source/de/anomic/plasma/plasmaCrawlEURL.java

@@ -109,7 +109,7 @@ public class plasmaCrawlEURL extends plasmaURL {
         return e;
     }
 
-    public synchronized Entry getEntry(String hash) {
+    public synchronized Entry getEntry(String hash) throws IOException {
 	return new Entry(hash);
     }
 
@@ -157,7 +157,7 @@ public class plasmaCrawlEURL extends plasmaURL {
 	    
 	}
 
-	public Entry(String hash) {
+	    public Entry(String hash) throws IOException {
             // generates an plasmaEURLEntry using the url hash
             // to speed up the access, the url-hashes are buffered
             // in the hash cache.
@@ -166,7 +166,6 @@ public class plasmaCrawlEURL extends plasmaURL {
             // - look into the filed properties
             // if the url cannot be found, this returns null
             this.hash = hash;
-	    try {
             byte[][] entry = urlHashCache.get(hash.getBytes());
             if (entry != null) {
                 this.referrer = new String(entry[1]);
@@ -181,7 +180,6 @@ public class plasmaCrawlEURL extends plasmaURL {
                 this.flags = new bitfield(entry[10]);
                 return;
             }
-	    } catch (Exception e) {}
         }
         
 	private void store() {
@@ -266,7 +264,11 @@ public class plasmaCrawlEURL extends plasmaURL {
             return i.hasNext();
         }
 	    public Object nextElement() {
+            try {
                 return new Entry(new String(((byte[][]) i.next())[0]));
+            } catch (IOException e) {
+                return null;
+            }
         }
     }
     

source/de/anomic/plasma/plasmaCrawlLURL.java

@@ -57,17 +57,13 @@ import java.io.IOException;
 import java.net.MalformedURLException;
 import java.net.URL;
 import java.text.SimpleDateFormat;
-import java.util.Collections;
 import java.util.Date;
-import java.util.HashSet;
 import java.util.Iterator;
 import java.util.LinkedList;
 import java.util.Locale;
 import java.util.Properties;
-import java.util.Set;
 
 import de.anomic.http.httpc;
-import de.anomic.kelondro.kelondroException;
 import de.anomic.kelondro.kelondroTree;
 import de.anomic.server.serverCodings;
 import de.anomic.server.serverObjects;
@@ -89,7 +85,7 @@ public final class plasmaCrawlLURL extends plasmaURL {
     private final LinkedList lcrawlResultStack; // 5 - local index: result of local crawling
     private final LinkedList gcrawlResultStack; // 6 - local index: triggered external
 
-    public static Set damagedURLS = Collections.synchronizedSet(new HashSet());
+    //public static Set damagedURLS = Collections.synchronizedSet(new HashSet());
     
     public plasmaCrawlLURL(File cachePath, int bufferkb) throws IOException {
         super();
@@ -173,7 +169,7 @@ public final class plasmaCrawlLURL extends plasmaURL {
         gcrawlResultStack.add(urlHash + initiatorHash + executorHash);
     }
 
-    public synchronized Entry getEntry(String hash) {
+    public synchronized Entry getEntry(String hash) throws IOException {
         return new Entry(hash);
     }
 
@@ -347,9 +343,9 @@ public final class plasmaCrawlLURL extends plasmaURL {
 //          serverLog.logFinest("PLASMA", "plasmaCrawlLURL/genTableProps initiatorHash=" + initiatorHash + " executorHash=" + executorHash);
             urlHash = getUrlHash(tabletype, i);
 //          serverLog.logFinest("PLASMA", "plasmaCrawlLURL/genTableProps urlHash=" + urlHash);
+            try {
                 urle = getEntry(urlHash);
 //              serverLog.logFinest("PLASMA", "plasmaCrawlLURL/genTableProps urle=" + urle.toString());
-            if (urle != null) try {
                 initiatorSeed = yacyCore.seedDB.getConnected(initiatorHash);
                 executorSeed = yacyCore.seedDB.getConnected(executorHash);
 
@@ -457,7 +453,7 @@ public final class plasmaCrawlLURL extends plasmaURL {
         store();        
     }
 
-    public Entry(String urlHash) {
+        public Entry(String urlHash) throws IOException {
             // generates an plasmaLURLEntry using the url hash
             // to speed up the access, the url-hashes are buffered
             // in the hash cache.
@@ -466,8 +462,8 @@ public final class plasmaCrawlLURL extends plasmaURL {
             // - look into the filed properties
             // if the url cannot be found, this returns null
             this.urlHash = urlHash;
-        try {
             byte[][] entry = plasmaCrawlLURL.this.urlHashCache.get(urlHash.getBytes());
+            try {
                 if (entry != null) {
                     this.url = new URL(new String(entry[1]).trim());
                     this.descr = (entry[2] == null) ? this.url.toString() : new String(entry[2]).trim();
@@ -484,12 +480,9 @@ public final class plasmaCrawlLURL extends plasmaURL {
                     this.snippet = null;
                     return;
                 }
-        } catch (MalformedURLException e) {
-            plasmaCrawlLURL.damagedURLS.add(this.urlHash);
-            System.out.println("DEBUG: Marked damaged Entry for removal (malformedURL). UrlHash: " + this.urlHash);
-            //serverLog.logSevere("PLASMA", "INTERNAL ERROR in plasmaLURL.entry/1: " + e.toString(), e);
             } catch (Exception e) {
                 serverLog.logSevere("PLASMA", "INTERNAL ERROR in plasmaLURL.entry/1: " + e.toString(), e);
+                throw new IOException("plasmaLURL.entry/1: " + e.toString());
             }
         }
 
@@ -742,14 +735,15 @@ public final class plasmaCrawlLURL extends plasmaURL {
             return i.hasNext();
         }
 
-        public Object next() {
-            try {
+        public Object next() throws RuntimeException {
             byte[] e = ((byte[][]) i.next())[0];
-                if (e == null) return null; else return new Entry(new String(e));
-            } catch (kelondroException e) {
-                e.printStackTrace();
-                error = true;
-                return null;
+            if (e == null) return null;
+            String hash = null;
+            try {
+                hash = new String(e);
+                return new Entry(hash);
+            } catch (IOException ex) {
+                throw new RuntimeException("error '" + ex.getMessage() + "' for hash " + hash);
             }
         }
         

source/de/anomic/plasma/plasmaCrawlNURL.java

@@ -363,7 +363,7 @@ public class plasmaCrawlNURL extends plasmaURL {
         }
     }
 
-    public synchronized Entry getEntry(String hash) {
+    public synchronized Entry getEntry(String hash) throws IOException {
         return new Entry(hash);
     }
 
@@ -431,7 +431,7 @@ public class plasmaCrawlNURL extends plasmaURL {
                return str.toString();
         }
 
-        public Entry(String hash) {
+        public Entry(String hash) throws IOException {
             // generates an plasmaNURLEntry using the url hash
             // to speed up the access, the url-hashes are buffered
             // in the hash cache.
@@ -440,9 +440,9 @@ public class plasmaCrawlNURL extends plasmaURL {
             // - look into the filed properties
             // if the url cannot be found, this returns null
             this.hash = hash;
-            try {
             byte[][] entry = urlHashCache.get(hash.getBytes());
             if (entry != null) {
+                //try {
                     this.initiator = new String(entry[1]);
                     this.url = new URL(new String(entry[2]).trim());
                     this.referrer = (entry[3] == null) ? dummyHash : new String(entry[3]);
@@ -455,11 +455,13 @@ public class plasmaCrawlNURL extends plasmaURL {
                     this.flags = new bitfield(entry[10]);
                     this.handle = Integer.parseInt(new String(entry[11]));
                     return;
+                //} catch (MalformedURLException e) {
+                //    throw new IOException("plasmaCrawlNURL/Entry: " + e);
+                //}
             } else {
                 // show that we found nothing
-                    this.url = null;
-                }
-            } catch (Exception e) {
+                throw new IOException("hash not found");
+                //this.url = null;
             }
         }
 

source/de/anomic/plasma/plasmaSwitchboard.java

@@ -1230,8 +1230,13 @@ public final class plasmaSwitchboard extends serverAbstractSwitch implements ser
             
             // create index
             String descr = document.getMainLongTitle();
+            String referrerHash;
+            try {
                 URL referrerURL = entry.referrerURL();
-            String referrerHash = (referrerURL == null) ? plasmaURL.dummyHash : plasmaURL.urlHash(referrerURL);
+                referrerHash = plasmaURL.urlHash(referrerURL);
+            } catch (IOException e) {
+                referrerHash = plasmaURL.dummyHash;
+            }
             String noIndexReason = "unspecified";
             if (processCase == 4) {
                 // proxy-load
@@ -1480,8 +1485,10 @@ public final class plasmaSwitchboard extends serverAbstractSwitch implements ser
         
         URL refererURL = null;
         String refererHash = urlEntry.referrerHash();
-        if ((refererHash != null) && (!refererHash.equals(plasmaURL.dummyHash))) {
+        if ((refererHash != null) && (!refererHash.equals(plasmaURL.dummyHash))) try {
             refererURL = this.urlPool.getURL(refererHash);
+        } catch (IOException e) {
+            refererURL = null;
         }
         cacheLoader.loadParallel(urlEntry.url(), urlEntry.name(), (refererURL!=null)?refererURL.toString():null, urlEntry.initiator(), urlEntry.depth(), profile);
         log.logInfo(stats + ": enqueued for load " + urlEntry.url() + " [" + urlEntry.hash() + "]");
@@ -1519,31 +1526,32 @@ public final class plasmaSwitchboard extends serverAbstractSwitch implements ser
         }
         
         // do the request
+        try {
             HashMap page = yacyClient.crawlOrder(remoteSeed, urlEntry.url(), urlPool.getURL(urlEntry.referrerHash()));
         
-        
             // check success
             /*
-         the result of the 'response' value can have one of the following values:
-         negative cases, no retry
-           denied      - the peer does not want to crawl that
-           exception   - an exception occurred
-         
-         negative case, retry possible
-           rejected    - the peer has rejected to process, but a re-try should be possible
-         
-         positive case with crawling
-           stacked     - the resource is processed asap
-         
-         positive case without crawling
-           double      - the resource is already in database, believed to be fresh and not reloaded
-                         the resource is also returned in lurl
+             * the result of the 'response' value can have one of the following
+             * values: negative cases, no retry denied - the peer does not want
+             * to crawl that exception - an exception occurred
+             * 
+             * negative case, retry possible rejected - the peer has rejected to
+             * process, but a re-try should be possible
+             * 
+             * positive case with crawling stacked - the resource is processed
+             * asap
+             * 
+             * positive case without crawling double - the resource is already
+             * in database, believed to be fresh and not reloaded the resource
+             * is also returned in lurl
              */
             if ((page == null) || (page.get("delay") == null)) {
                 log.logInfo("CRAWL: REMOTE CRAWL TO PEER " + remoteSeed.getName() + " FAILED. CAUSE: unknown (URL=" + urlEntry.url().toString() + ")");
-            if (remoteSeed != null) yacyCore.peerActions.peerDeparture(remoteSeed);
+                if (remoteSeed != null)
+                    yacyCore.peerActions.peerDeparture(remoteSeed);
                 return false;
-        } else try {
+            } else
+                try {
                     log.logFine("plasmaSwitchboard.processRemoteCrawlTrigger: remoteSeed=" + remoteSeed.getName() + ", url=" + urlEntry.url().toString() + ", response=" + page.toString()); // DEBUG
 
                     int newdelay = Integer.parseInt((String) page.get("delay"));
@@ -1556,9 +1564,7 @@ public final class plasmaSwitchboard extends serverAbstractSwitch implements ser
                         String lurl = (String) page.get("lurl");
                         if ((lurl != null) && (lurl.length() != 0)) {
                             String propStr = crypt.simpleDecode(lurl, (String) page.get("key"));
-                    plasmaCrawlLURL.Entry entry = urlPool.loadedURL.addEntry(
-                    urlPool.loadedURL.newEntry(propStr, true),
-                    yacyCore.seedDB.mySeed.hash, remoteSeed.hash, 1);
+                            plasmaCrawlLURL.Entry entry = urlPool.loadedURL.addEntry(urlPool.loadedURL.newEntry(propStr, true), yacyCore.seedDB.mySeed.hash, remoteSeed.hash, 1);
                             urlPool.noticeURL.remove(entry.hash());
                             log.logInfo(STR_REMOTECRAWLTRIGGER + remoteSeed.getName() + " SUPERFLUOUS. CAUSE: " + page.get("reason") + " (URL=" + urlEntry.url().toString() + "). URL IS CONSIDERED AS 'LOADED!'");
                             return true;
@@ -1575,6 +1581,10 @@ public final class plasmaSwitchboard extends serverAbstractSwitch implements ser
                     log.logSevere(STR_REMOTECRAWLTRIGGER + remoteSeed.getName() + " FAILED. CLIENT RETURNED: " + page.toString(), e);
                     return false;
                 }
+        } catch (IOException e) {
+            log.logSevere(STR_REMOTECRAWLTRIGGER + remoteSeed.getName() + " FAILED. URL CANNOT BE RETRIEVED from referrer hash: " + urlEntry.referrerHash(), e);
+            return false;
+        }
     }
     
     private static SimpleDateFormat DateFormatter = new SimpleDateFormat("EEE, dd MMM yyyy");
@@ -1825,11 +1835,11 @@ public final class plasmaSwitchboard extends serverAbstractSwitch implements ser
     }
     
     // method for index deletion
-    public int removeAllUrlReferences(URL url, boolean fetchOnline) {
+    public int removeAllUrlReferences(URL url, boolean fetchOnline) throws IOException {
         return removeAllUrlReferences(plasmaURL.urlHash(url), fetchOnline);
     }
     
-    public int removeAllUrlReferences(String urlhash, boolean fetchOnline) {
+    public int removeAllUrlReferences(String urlhash, boolean fetchOnline) throws IOException {
         // find all the words in a specific resource and remove the url reference from every word index
         // finally, delete the url entry
         

source/de/anomic/plasma/plasmaSwitchboardQueue.java

@@ -273,7 +273,7 @@ public class plasmaSwitchboardQueue {
             return responseHeader;
         }
 
-        public URL referrerURL() {
+        public URL referrerURL() throws IOException {
             if (referrerURL == null) {
                 if ((referrerHash == null) || (referrerHash.equals(plasmaURL.dummyHash))) return null;
                 referrerURL = lurls.getEntry(referrerHash).url();

source/de/anomic/plasma/plasmaURLPool.java

@@ -71,7 +71,7 @@ public class plasmaURLPool {
         return null;
     }
     
-    public URL getURL(String urlhash) {
+    public URL getURL(String urlhash) throws IOException {
         if (urlhash.equals(plasmaURL.dummyHash)) return null;
         plasmaCrawlNURL.Entry ne = noticeURL.getEntry(urlhash);
         if (ne != null) return ne.url();

source/yacy.java

@@ -1180,19 +1180,22 @@ public final class yacy {
     private static void urldbcleanup(String homePath) {
         File root = new File(homePath);
         File dbroot = new File(root, "DATA/PLASMADB");
+        HashSet damagedURLS = new HashSet();
         try {
             plasmaCrawlLURL currentUrlDB = new plasmaCrawlLURL(new File(dbroot, "urlHash.db"), 4194304);
             Iterator eiter = currentUrlDB.entries(true, false);
             int iteratorCount = 0;
-            while (eiter.hasNext()) {
+            while (eiter.hasNext()) try {
                 eiter.next();
                 iteratorCount++;
+            } catch (RuntimeException e) {
+                String m = e.getMessage();
+                damagedURLS.add(m.substring(m.length() - 12));
             }
             try { Thread.sleep(1000); } catch (InterruptedException e) { }
-            System.out.println("URLs vorher: " + currentUrlDB.size() + " Entries loaded during Iteratorloop: " + iteratorCount + " kaputte URLs: " + plasmaCrawlLURL.damagedURLS.size());
-            synchronized(plasmaCrawlLURL.damagedURLS)
-            {
-                Iterator eiter2 = plasmaCrawlLURL.damagedURLS.iterator();
+            System.out.println("URLs vorher: " + currentUrlDB.size() + " Entries loaded during Iteratorloop: " + iteratorCount + " kaputte URLs: " + damagedURLS.size());
+
+            Iterator eiter2 = damagedURLS.iterator();
             String urlHash;
             while (eiter2.hasNext()) {
                 urlHash = (String) eiter2.next();
@@ -1220,20 +1223,15 @@ public final class yacy {
                         if (res.statusCode == 200) {
                             entry[1] = newUrl.toString().getBytes();
                             currentUrlDB.urlHashCache.put(entry);
-                                System.out.println("UrlDB-Entry with urlHash '" + urlHash + 
-                                        "' corrected\n\tURL: " + oldUrlStr + " -> " + newUrlStr);
+                            System.out.println("UrlDB-Entry with urlHash '" + urlHash + "' corrected\n\tURL: " + oldUrlStr + " -> " + newUrlStr);
                         } else {
                             currentUrlDB.remove(urlHash);
-                                System.out.println("UrlDB-Entry with urlHash '" + urlHash +
-                                        "' removed\n\tURL: " + oldUrlStr +
-                                        "\n\tConnection Status: " + res.status);
+                            System.out.println("UrlDB-Entry with urlHash '" + urlHash + "' removed\n\tURL: " + oldUrlStr + "\n\tConnection Status: " + res.status);
                         }
                     }
                 } catch (Exception e) {
                     currentUrlDB.remove(urlHash);
-                        System.out.println("UrlDB-Entry with urlHash '" + urlHash + 
-                                           "' removed\n\tURL: " + oldUrlStr + 
-                                           "\n\tExecption: " + e.getMessage());
+                    System.out.println("UrlDB-Entry with urlHash '" + urlHash + "' removed\n\tURL: " + oldUrlStr + "\n\tExecption: " + e.getMessage());
                 } finally {
                     if (theHttpc != null) try {
                         theHttpc.close();
@@ -1241,9 +1239,8 @@ public final class yacy {
                     } catch (Exception e) { }
                 }
             }
-            }
-            plasmaCrawlLURL.damagedURLS.clear();
-            System.out.println("URLs nachher: " + currentUrlDB.size() + " kaputte URLs: " + plasmaCrawlLURL.damagedURLS.size());
+
+            System.out.println("URLs nachher: " + currentUrlDB.size() + " kaputte URLs: " + damagedURLS.size());
             currentUrlDB.close();
         } catch (IOException e) {
             e.printStackTrace();
@@ -1254,13 +1251,20 @@ public final class yacy {
      * Main-method which is started by java. Checks for special arguments or
      * starts up the application.
      * 
-    * @param args Given arguments from the command line.
+     * @param args
+     *            Given arguments from the command line.
      */
     public static void main(String args[]) {
 
         // check memory amount
         System.gc();
-        long startupMemFree  = Runtime.getRuntime().freeMemory(); // the amount of free memory in the Java Virtual Machine
+        long startupMemFree  = Runtime.getRuntime().freeMemory(); // the
+                                                                    // amount of
+                                                                    // free
+                                                                    // memory in
+                                                                    // the Java
+                                                                    // Virtual
+                                                                    // Machine
         long startupMemTotal = Runtime.getRuntime().totalMemory(); // the total amount of memory in the Java virtual machine; may vary over time
 
         // go into headless awt mode