NoIssue - Added filterhit statistics loading capability

abp/stats/filterhits.py

 # This file is part of Adblock Plus <https://adblockplus.org/>,
 # Copyright (C) 2006-present eyeo GmbH
 #
 # Adblock Plus is free software: you can redistribute it and/or modify
 # it under the terms of the GNU General Public License version 3 as
 # published by the Free Software Foundation.
 #
 # Adblock Plus is distributed in the hope that it will be useful,
 # but WITHOUT ANY WARRANTY; without even the implied warranty of
 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
@@ skipping 17 matching lines @@
     sources: iterable of str
         With the filter sources we're interested in. If not None, only filters
         from these sources will be included in the result.
 
     Returns
     -------
     generator of dict
         With the csv entries.
 
     """
-    integer_cols = ['onehour_sessions', 'hits']

[Vasily Kuznetsov, 2019/04/03 16:36:28]

What about domains and rootdomains? They seem useful too.

+    integer_cols = ['onehour_sessions', 'hits', 'domains', 'rootdomains']
 
     with open(path) as csvstream:
         reader = csv.DictReader(csvstream)
 
         for entry in reader:
             if sources and entry['source'] not in sources:
                 continue
             for col in integer_cols:
-                entry[col] = int(entry[col])
+                try:
+                    entry[col] = int(entry[col])
+                except KeyError:
+                    continue
 
             yield entry