HiranChaudhuri commented on code in PR #829:
URL: https://github.com/apache/nutch/pull/829#discussion_r1801520902
##########
src/java/org/apache/nutch/crawl/Injector.java:
##########
@@ -402,40 +402,38 @@ public void inject(Path crawlDb, Path urlDir, boolean
overwrite,
// lock an existing crawldb to prevent multiple simultaneous updates
Path lock = CrawlDb.lock(conf, crawlDb, false);
Review Comment:
My understanding of the code is not good enough to take that decision, so I
went for the safe side to ensure the DB is already locked when the job is
preparing. Am ok with that change. :-)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscr...@nutch.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
