jenkins-bot has submitted this change and it was merged.
Change subject: Sort list of CSVs before iterating over them for projectcount
aggregation
......................................................................
Sort list of CSVs before iterating over them for projectcount aggregation
Having the list of CSVs sorted gives a better feeling of how much work
is still to be done for a given day in verbose mode, which prints the
CSV names.
Change-Id: Ieaffc02e79aa60f13c8af5933bb4286091d7be77
---
M aggregator/projectcounts.py
1 file changed, 2 insertions(+), 1 deletion(-)
Approvals:
Mforns: Looks good to me, approved
jenkins-bot: Verified
diff --git a/aggregator/projectcounts.py b/aggregator/projectcounts.py
index 76b0b3c..230b2ec 100644
--- a/aggregator/projectcounts.py
+++ b/aggregator/projectcounts.py
@@ -145,7 +145,8 @@
:param force_recomputation: If True, recompute data for the given days,
even if it is already in the CSV. (Default: False)
"""
- for csv_file_abs in glob.glob(os.path.join(target_dir_abs, '*.csv')):
+ for csv_file_abs in sorted(glob.glob(os.path.join(
+ target_dir_abs, '*.csv'))):
logging.info("Updating csv '%s'" % (csv_file_abs))
csv_data = {}
--
To view, visit https://gerrit.wikimedia.org/r/173257
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ieaffc02e79aa60f13c8af5933bb4286091d7be77
Gerrit-PatchSet: 1
Gerrit-Project: analytics/aggregator
Gerrit-Branch: master
Gerrit-Owner: QChris <[email protected]>
Gerrit-Reviewer: Mforns <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
