x/stackalytics
Code Issues Proposed changes

Null values handling

Browse Source 
Users with null id should not be stored into runtime storage.

Change-Id: I261dec406c6e175015a5e1203c5281aac4734ded
This commit is contained in:
Ilya Shakhat 2013-10-22 15:24:35 +04:00
parent 96dc8b7e2d
commit 7c5b3498ac
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
stackalytics/processor/utils.py
View File

@ -81,6 +81,7 @@ def make_range(start, stop, step):
def store_user(runtime_storage_inst, user): def store_user(runtime_storage_inst, user):
if user.get('user_id'):
runtime_storage_inst.set_by_key('user:%s' % user['user_id'], user) runtime_storage_inst.set_by_key('user:%s' % user['user_id'], user)
if user.get('launchpad_id'): if user.get('launchpad_id'):
runtime_storage_inst.set_by_key('user:%s' % user['launchpad_id'], user) runtime_storage_inst.set_by_key('user:%s' % user['launchpad_id'], user)
@ -89,7 +90,9 @@ def store_user(runtime_storage_inst, user):
def load_user(runtime_storage_inst, user_id): def load_user(runtime_storage_inst, user_id):
if user_id:
return runtime_storage_inst.get_by_key('user:%s' % user_id) return runtime_storage_inst.get_by_key('user:%s' % user_id)
return None
def load_repos(runtime_storage_inst): def load_repos(runtime_storage_inst):