Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Issue 8867001: Reports - cleaning users (Closed)

Created:
Nov. 16, 2012, 8:30 a.m. by Andrey Novikov
Modified:
Nov. 16, 2012, 8:39 a.m.
Reviewers:
Wladimir Palant
Visibility:
Public.

Description

Reports - cleaning users

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+32 lines, -0 lines) Patch
A sitescripts/reports/bin/removeOldUsers.py View 1 chunk +31 lines, -0 lines 1 comment Download
M sitescripts/reports/schema.sql View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 2
Andrey Novikov
Nov. 16, 2012, 8:30 a.m. (2012-11-16 08:30:03 UTC) #1
Wladimir Palant
Nov. 16, 2012, 8:34 a.m. (2012-11-16 08:34:50 UTC) #2
LGTM with the comment addressed.

http://codereview.adblockplus.org/8867001/diff/1/sitescripts/reports/bin/remo...
File sitescripts/reports/bin/removeOldUsers.py (right):

http://codereview.adblockplus.org/8867001/diff/1/sitescripts/reports/bin/remo...
sitescripts/reports/bin/removeOldUsers.py:18: import MySQLdb
Doesn't look like you actually need that package.

Powered by Google App Engine
This is Rietveld