[ 
https://issues.apache.org/jira/browse/NUTCH-2375?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16060760#comment-16060760
 ] 

ASF GitHub Bot commented on NUTCH-2375:
---------------------------------------

Omkar20895 commented on a change in pull request #188: NUTCH-2375 Upgrade the 
code base from org.apache.hadoop.mapred to org.apache.hadoop.mapreduce
URL: https://github.com/apache/nutch/pull/188#discussion_r123726840
 
 

 ##########
 File path: src/java/org/apache/nutch/crawl/CrawlDb.java
 ##########
 @@ -111,7 +116,7 @@ public void update(Path crawlDb, Path[] segments, boolean 
normalize,
       LOG.info("CrawlDb update: Merging segment data into db.");
     }
     try {
-      JobClient.runJob(job);
+      int complete = job.waitForCompletion(true)?0:1;
     } catch (IOException e) {
 
 Review comment:
   @sebastian-nagel in the above job when there is an IOException the output 
path is being removed (cleaning up) before throwing up the error and the same 
needs to be done when we catch the other two exceptions i.e 
ClassNotFoundException and InterruptedException also, isn't it? Please feel 
free to correct me if I am wrong. 
 
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


> Upgrade the code base from org.apache.hadoop.mapred to 
> org.apache.hadoop.mapreduce
> ----------------------------------------------------------------------------------
>
>                 Key: NUTCH-2375
>                 URL: https://issues.apache.org/jira/browse/NUTCH-2375
>             Project: Nutch
>          Issue Type: Improvement
>          Components: deployment
>            Reporter: Omkar Reddy
>
> Nutch is still using the deprecated org.apache.hadoop.mapred dependency which 
> has been deprecated. It need to be updated to org.apache.hadoop.mapreduce 
> dependency. 



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to