Author: kwright
Date: Mon Oct 20 11:42:48 2014
New Revision: 1633109

URL: http://svn.apache.org/r1633109
Log:
Pull up more CONNECTORS-1077-related work from trunk.

Modified:
    manifoldcf/branches/dev_1x/   (props changed)
    
manifoldcf/branches/dev_1x/connectors/filesystem/connector/src/main/java/org/apache/manifoldcf/agents/output/filesystem/FileOutputConnector.java
    
manifoldcf/branches/dev_1x/connectors/hdfs/connector/src/main/java/org/apache/manifoldcf/agents/output/hdfs/HDFSOutputConnector.java

Propchange: manifoldcf/branches/dev_1x/
------------------------------------------------------------------------------
  Merged /manifoldcf/trunk:r1633062-1633063

Modified: 
manifoldcf/branches/dev_1x/connectors/filesystem/connector/src/main/java/org/apache/manifoldcf/agents/output/filesystem/FileOutputConnector.java
URL: 
http://svn.apache.org/viewvc/manifoldcf/branches/dev_1x/connectors/filesystem/connector/src/main/java/org/apache/manifoldcf/agents/output/filesystem/FileOutputConnector.java?rev=1633109&r1=1633108&r2=1633109&view=diff
==============================================================================
--- 
manifoldcf/branches/dev_1x/connectors/filesystem/connector/src/main/java/org/apache/manifoldcf/agents/output/filesystem/FileOutputConnector.java
 (original)
+++ 
manifoldcf/branches/dev_1x/connectors/filesystem/connector/src/main/java/org/apache/manifoldcf/agents/output/filesystem/FileOutputConnector.java
 Mon Oct 20 11:42:48 2014
@@ -215,7 +215,7 @@ public class FileOutputConnector extends
           }
           else
           {
-              
activities.recordActivity(null,INGEST_ACTIVITY,null,documentURI,activities.CREATED_DIRECTORY,"Couldn't
 create directory ("+newPath+").");
+              
activities.recordActivity(null,INGEST_ACTIVITY,null,documentURI,activities.CREATED_DIRECTORY,"Couldn't
 create directory. '"+newPath+"'");
               throw new ManifoldCFException("Could not create directory 
'"+newPath+"'.  Permission issue?");
           }
         }
@@ -247,7 +247,7 @@ public class FileOutputConnector extends
             continue;
           }
           // Probably some other error
-          activities.recordActivity(null, INGEST_ACTIVITY, new 
Long(document.getBinaryLength()), documentURI, activities.EXCEPTION, "Rejected 
due to FileNotFoundException.");
+          activities.recordActivity(null, INGEST_ACTIVITY, new 
Long(document.getBinaryLength()), documentURI, activities.EXCEPTION, 
"Unexpected exception while creating the file.'" + outputPath + "'");
           throw new ManifoldCFException("Could not create file 
'"+outputPath+"': "+e.getMessage(),e);
         }
       }
@@ -259,7 +259,7 @@ public class FileOutputConnector extends
         FileChannel channel = output.getChannel();
         FileLock lock = channel.tryLock();
         if (lock == null){
-          activities.recordActivity(null,INGEST_ACTIVITY, new 
Long(document.getBinaryLength()), documentURI,activities.EXCEPTION,"Rejected 
due to ServiceInterruptionException.");
+          activities.recordActivity(null,INGEST_ACTIVITY, new 
Long(document.getBinaryLength()), documentURI,activities.EXCEPTION,"Could not 
lock file: '" + outputPath + "'");
           throw new ServiceInterruption("Could not lock file: 
'"+outputPath+"'",null,1000L,-1L,10,false);
         }
 
@@ -292,19 +292,19 @@ public class FileOutputConnector extends
         }
       }
     } catch (URISyntaxException e) {
-      activities.recordActivity(null,INGEST_ACTIVITY,new 
Long(document.getBinaryLength()), documentURI,activities.EXCEPTION,"Rejected 
due to URISyntaxException");
+      activities.recordActivity(null,INGEST_ACTIVITY,new 
Long(document.getBinaryLength()), documentURI,activities.EXCEPTION,"Failed to 
write document due to: " + e.getMessage());
       handleURISyntaxException(e);
       return DOCUMENTSTATUS_REJECTED;
     } catch (FileNotFoundException e) {
-      activities.recordActivity(null,INGEST_ACTIVITY,new 
Long(document.getBinaryLength()), documentURI,activities.EXCEPTION,"Rejected 
due to FileNotFoundException");
+      activities.recordActivity(null,INGEST_ACTIVITY,new 
Long(document.getBinaryLength()), documentURI,activities.EXCEPTION,"Failed to 
write document due to: " + e.getMessage());
       handleFileNotFoundException(e);
       return DOCUMENTSTATUS_REJECTED;
     } catch (SecurityException e) {
-      activities.recordActivity(null,INGEST_ACTIVITY,new 
Long(document.getBinaryLength()), documentURI,activities.EXCEPTION,"Rejected 
due to SecurityException");
+      activities.recordActivity(null,INGEST_ACTIVITY,new 
Long(document.getBinaryLength()), documentURI,activities.EXCEPTION,"Failed to 
write document due to: " + e.getMessage());
       handleSecurityException(e);
       return DOCUMENTSTATUS_REJECTED;
     } catch (IOException e) {
-      activities.recordActivity(null,INGEST_ACTIVITY,new 
Long(document.getBinaryLength()), documentURI ,activities.EXCEPTION,"Rejected 
due to IOException.");
+      activities.recordActivity(null,INGEST_ACTIVITY,new 
Long(document.getBinaryLength()), documentURI ,activities.EXCEPTION,"Failed to 
write document due to: " + e.getMessage());
       handleIOException(e);
       return DOCUMENTSTATUS_REJECTED;
     }
@@ -432,23 +432,23 @@ public class FileOutputConnector extends
         catch (FileNotFoundException e)
         {
           // Probably some other error
-          
activities.recordActivity(null,REMOVE_ACTIVITY,null,documentURI,activities.EXCEPTION,"Couldn't
 deleted due to FileNotFoundException");
+          
activities.recordActivity(null,REMOVE_ACTIVITY,null,documentURI,activities.EXCEPTION,"Couldn't
 delete the file due to FileNotFoundException '"+outputPath+"':" + 
e.getMessage());
           throw new ManifoldCFException("Could not zero out file 
'"+outputPath+"': "+e.getMessage(),e);
         }
       }
       // Just close it, to make a zero-length grave marker.
       output.close();
     } catch (URISyntaxException e) {
-      activities.recordActivity(null,REMOVE_ACTIVITY,null, 
documentURI,activities.EXCEPTION,"Rejected due to URISyntaxException");
+      activities.recordActivity(null,REMOVE_ACTIVITY,null, 
documentURI,activities.EXCEPTION,"Failed to delete document due to: " + 
e.getMessage());
       handleURISyntaxException(e);
     } catch (FileNotFoundException e) {
-      activities.recordActivity(null,REMOVE_ACTIVITY,null, 
documentURI,activities.EXCEPTION,"Rejected due to FileNotFoundException");
+      activities.recordActivity(null,REMOVE_ACTIVITY,null, 
documentURI,activities.EXCEPTION,"Failed to delete document due to: " + 
e.getMessage());
       handleFileNotFoundException(e);
     } catch (SecurityException e) {
-      activities.recordActivity(null,REMOVE_ACTIVITY,null, 
documentURI,activities.EXCEPTION,"Rejected due to SecurityException");
+      activities.recordActivity(null,REMOVE_ACTIVITY,null, 
documentURI,activities.EXCEPTION,"Failed to delete document due to: " + 
e.getMessage());
       handleSecurityException(e);
     } catch (IOException e) {
-      activities.recordActivity(null,INGEST_ACTIVITY,null, documentURI 
,activities.EXCEPTION,"Rejected due to IOException.");
+      activities.recordActivity(null,REMOVE_ACTIVITY,null, documentURI 
,activities.EXCEPTION,"Failed to delete document due to: " + e.getMessage());
       handleIOException(e);
     }
 

Modified: 
manifoldcf/branches/dev_1x/connectors/hdfs/connector/src/main/java/org/apache/manifoldcf/agents/output/hdfs/HDFSOutputConnector.java
URL: 
http://svn.apache.org/viewvc/manifoldcf/branches/dev_1x/connectors/hdfs/connector/src/main/java/org/apache/manifoldcf/agents/output/hdfs/HDFSOutputConnector.java?rev=1633109&r1=1633108&r2=1633109&view=diff
==============================================================================
--- 
manifoldcf/branches/dev_1x/connectors/hdfs/connector/src/main/java/org/apache/manifoldcf/agents/output/hdfs/HDFSOutputConnector.java
 (original)
+++ 
manifoldcf/branches/dev_1x/connectors/hdfs/connector/src/main/java/org/apache/manifoldcf/agents/output/hdfs/HDFSOutputConnector.java
 Mon Oct 20 11:42:48 2014
@@ -290,7 +290,7 @@ public class HDFSOutputConnector extends
       activities.recordActivity(startTime, INGEST_ACTIVITY, new 
Long(document.getBinaryLength()), documentURI, "OK", null);
       return DOCUMENTSTATUS_ACCEPTED;
     } catch (URISyntaxException e) {
-      activities.recordActivity(null,INGEST_ACTIVITY,new 
Long(document.getBinaryLength()),documentURI,activities.EXCEPTION,"Rejected due 
to URISyntaxException");
+      activities.recordActivity(null,INGEST_ACTIVITY,new 
Long(document.getBinaryLength()),documentURI,activities.EXCEPTION,"Failed to 
write document due to: " + e.getMessage());
       handleURISyntaxException(e);
       return DOCUMENTSTATUS_REJECTED;
     }
@@ -324,10 +324,10 @@ public class HDFSOutputConnector extends
       deleteFile(path,activities,documentURI);
       activities.recordActivity(startTime, REMOVE_ACTIVITY, null, documentURI, 
"OK", null);
     } catch (JSONException e) {
-      
activities.recordActivity(null,REMOVE_ACTIVITY,null,documentURI,activities.EXCEPTION,"Rejected
 due to JSONException.");
+      
activities.recordActivity(null,REMOVE_ACTIVITY,null,documentURI,activities.EXCEPTION,"Failed
 to delete document due to: " + e.getMessage());
       handleJSONException(e);
     } catch (URISyntaxException e) {
-      
activities.recordActivity(null,REMOVE_ACTIVITY,null,documentURI,activities.EXCEPTION,"Rejected
 due to URISyntaxException.");
+      
activities.recordActivity(null,REMOVE_ACTIVITY,null,documentURI,activities.EXCEPTION,"Failed
 to delete document due to: " + e.getMessage());
       handleURISyntaxException(e);
     }
   }
@@ -680,17 +680,17 @@ public class HDFSOutputConnector extends
       t.finishUp();
     } catch (InterruptedException e) {
       t.interrupt();
-      
activities.recordActivity(null,INGEST_ACTIVITY,null,documentURI,activities.EXCEPTION,"Rejected
 due to InterruptedException.");
+      
activities.recordActivity(null,INGEST_ACTIVITY,null,documentURI,activities.EXCEPTION,"Failed
 to write document due to: " + e.getMessage());
       throw new ManifoldCFException("Interrupted: 
"+e.getMessage(),e,ManifoldCFException.INTERRUPTED);
     } catch (java.net.SocketTimeoutException e) {
-      
activities.recordActivity(null,INGEST_ACTIVITY,null,documentURI,activities.EXCEPTION,"Rejected
 due to SocketTimeoutException.");
+      
activities.recordActivity(null,INGEST_ACTIVITY,null,documentURI,activities.EXCEPTION,"Failed
 to write document due to: " + e.getMessage());
       handleIOException(e);
     } catch (InterruptedIOException e) {
       t.interrupt();
-      
activities.recordActivity(null,INGEST_ACTIVITY,null,documentURI,activities.EXCEPTION,"Rejected
 due to InterruptedIOException.");
+      
activities.recordActivity(null,INGEST_ACTIVITY,null,documentURI,activities.EXCEPTION,"Failed
 to write document due to: " + e.getMessage());
       handleIOException(e);
     } catch (IOException e) {
-      
activities.recordActivity(null,INGEST_ACTIVITY,null,documentURI,activities.EXCEPTION,"Rejected
 due to IOException.");
+      
activities.recordActivity(null,INGEST_ACTIVITY,null,documentURI,activities.EXCEPTION,"Failed
 to write document due to: " + e.getMessage());
       handleIOException(e);
     }
   }
@@ -739,17 +739,17 @@ public class HDFSOutputConnector extends
       t.finishUp();
     } catch (InterruptedException e) {
       t.interrupt();
-      
activities.recordActivity(null,REMOVE_ACTIVITY,null,documentURI,activities.EXCEPTION,"Rejected
 due to InterruptedException.");
+      
activities.recordActivity(null,REMOVE_ACTIVITY,null,documentURI,activities.EXCEPTION,"Failed
 to delete document due to: " + e.getMessage());
       throw new ManifoldCFException("Interrupted: 
"+e.getMessage(),e,ManifoldCFException.INTERRUPTED);
     } catch (java.net.SocketTimeoutException e) {
-      
activities.recordActivity(null,REMOVE_ACTIVITY,null,documentURI,activities.EXCEPTION,"Rejected
 due to SocketTimeoutException.");
+      
activities.recordActivity(null,REMOVE_ACTIVITY,null,documentURI,activities.EXCEPTION,"Failed
 to delete document due to: " + e.getMessage());
       handleIOException(e);
     } catch (InterruptedIOException e) {
       t.interrupt();
-      
activities.recordActivity(null,REMOVE_ACTIVITY,null,documentURI,activities.EXCEPTION,"Rejected
 due to InterruptedIOException.");
+      
activities.recordActivity(null,REMOVE_ACTIVITY,null,documentURI,activities.EXCEPTION,"Failed
 to delete document due to: " + e.getMessage());
       handleIOException(e);
     } catch (IOException e) {
-      
activities.recordActivity(null,REMOVE_ACTIVITY,null,documentURI,activities.EXCEPTION,"Rejected
 due to IOException.");
+      
activities.recordActivity(null,REMOVE_ACTIVITY,null,documentURI,activities.EXCEPTION,"Failed
 to delete document due to: " + e.getMessage());
       handleIOException(e);
     }
   }


Reply via email to