From 31a375bde738a5c9c0a4930075be4535ffff6baf Mon Sep 17 00:00:00 2001 From: Pablo Hoffman Date: Sun, 10 Jul 2011 04:18:50 -0300 Subject: [PATCH] Close the scheduler after closing the scraper and downloader. This shouldn't have any real effect in practice, but it feels more appropiate to close the components in this order --- scrapy/core/engine.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/scrapy/core/engine.py b/scrapy/core/engine.py index 959f035b9..6f9f41252 100644 --- a/scrapy/core/engine.py +++ b/scrapy/core/engine.py @@ -271,15 +271,15 @@ class ExecutionEngine(object): dfd = slot.close() - dfd.addBoth(lambda _: self.scheduler.close_spider(spider)) - dfd.addErrback(log.err, spider=spider) - dfd.addBoth(lambda _: self.downloader.close_spider(spider)) dfd.addErrback(log.err, spider=spider) dfd.addBoth(lambda _: self.scraper.close_spider(spider)) dfd.addErrback(log.err, spider=spider) + dfd.addBoth(lambda _: self.scheduler.close_spider(spider)) + dfd.addErrback(log.err, spider=spider) + dfd.addBoth(lambda _: self._cancel_next_call(spider)) dfd.addErrback(log.err, spider=spider)