You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@manifoldcf.apache.org by kw...@apache.org on 2012/08/09 00:54:55 UTC
svn commit: r1371007 -
/manifoldcf/branches/CONNECTORS-501/framework/pull-agent/src/main/java/org/apache/manifoldcf/crawler/jobs/IntrinsicLink.java
Author: kwright
Date: Wed Aug 8 22:54:55 2012
New Revision: 1371007
URL: http://svn.apache.org/viewvc?rev=1371007&view=rev
Log:
Don't clean up IntrinsicLink table either
Modified:
manifoldcf/branches/CONNECTORS-501/framework/pull-agent/src/main/java/org/apache/manifoldcf/crawler/jobs/IntrinsicLink.java
Modified: manifoldcf/branches/CONNECTORS-501/framework/pull-agent/src/main/java/org/apache/manifoldcf/crawler/jobs/IntrinsicLink.java
URL: http://svn.apache.org/viewvc/manifoldcf/branches/CONNECTORS-501/framework/pull-agent/src/main/java/org/apache/manifoldcf/crawler/jobs/IntrinsicLink.java?rev=1371007&r1=1371006&r2=1371007&view=diff
==============================================================================
--- manifoldcf/branches/CONNECTORS-501/framework/pull-agent/src/main/java/org/apache/manifoldcf/crawler/jobs/IntrinsicLink.java (original)
+++ manifoldcf/branches/CONNECTORS-501/framework/pull-agent/src/main/java/org/apache/manifoldcf/crawler/jobs/IntrinsicLink.java Wed Aug 8 22:54:55 2012
@@ -354,24 +354,9 @@ public class IntrinsicLink extends org.a
performDelete(sb.toString(),list,null);
noteModifications(0,0,1);
-
- // Delete matches for parentIDHashField
- sb = new StringBuilder("WHERE ");
- list = new ArrayList();
-
- sb.append("EXISTS(SELECT 'x' FROM ").append(joinTableName).append(" WHERE ")
- .append(buildConjunctionClause(list,new ClauseDescription[]{
- new UnitaryClause(joinTableJobColumn,jobID),
- new JoinClause(joinTableIDColumn,getTableName()+"."+parentIDHashField)})).append(" AND ");
-
- sb.append(joinTableCriteria);
- list.addAll(joinTableParams);
-
- sb.append(")");
-
- performDelete(sb.toString(),list,null);
- noteModifications(0,0,1);
-
+
+ // DON'T delete ParentID matches; we need to leave those around for bookkeeping to
+ // be correct. See CONNECTORS-501.
}
catch (ManifoldCFException e)
{
@@ -451,14 +436,8 @@ public class IntrinsicLink extends org.a
new UnitaryClause(jobIDField,jobID),
new MultiClause(childIDHashField,list)}));
performDelete(sb.toString(),thisList,null);
-
- sb = new StringBuilder("WHERE ");
- thisList = new ArrayList();
-
- sb.append(buildConjunctionClause(thisList,new ClauseDescription[]{
- new UnitaryClause(jobIDField,jobID),
- new MultiClause(parentIDHashField,list)}));
- performDelete(sb.toString(),thisList,null);
+
+ // DON'T do parentID matches; we need to leave those around. See CONNECTORS-501.
}
/** Remove all target links of the specified source documents that are not marked as "new" or "existing", and