Remove overall_statistics as they are no longer used

so1us2 · Jan 22, 2020 · a5abb6d · a5abb6d
1 parent c0aadfc
commit a5abb6d
Showing 1 changed file with 0 additions and 25 deletions.
diff --git a/ckanext/harvest/logic/dictization.py b/ckanext/harvest/logic/dictization.py
@@ -117,14 +117,10 @@ def _get_source_status(source, context):
 
     job_count = HarvestJob.filter(source=source).count()
 
-    # Overall statistics are commented out because of timeout issues
-    # an issue has been raised https://github.com/ckan/ckanext-harvest/issues/366
-    # for the CKAN team to look into
     out = {
         'job_count': 0,
         'next_harvest': '',
         'last_harvest_request': '',
-        # 'overall_statistics': {'added': 0, 'errors': 0},
         }
 
     if not job_count:
@@ -145,28 +141,7 @@ def _get_source_status(source, context):
         .order_by(HarvestJob.created.desc()).first()
 
     if last_job:
-        # TODO: Should we encode the dates as strings?
         out['last_harvest_request'] = str(last_job.gather_finished)
-
-        # Overall statistics
-        # packages = model.Session.query(distinct(HarvestObject.package_id),
-        #                                Package.name) \
-        #     .join(Package).join(HarvestSource) \
-        #     .filter(HarvestObject.source == source) \
-        #     .filter(
-        #     HarvestObject.current == True  # noqa: E711
-        # ).filter(Package.state == u'active')
-
-        # out['overall_statistics']['added'] = packages.count()
-
-        # gather_errors = model.Session.query(HarvestGatherError) \
-        #     .join(HarvestJob).join(HarvestSource) \
-        #     .filter(HarvestJob.source == source).count()
-
-        # object_errors = model.Session.query(HarvestObjectError) \
-        #     .join(HarvestObject).join(HarvestJob).join(HarvestSource) \
-        #     .filter(HarvestJob.source == source).count()
-        # out['overall_statistics']['errors'] = gather_errors + object_errors
     else:
         out['last_harvest_request'] = 'Not yet harvested'