x/stackalytics
Code Issues Proposed changes

Merge "Rename update_user() as an internal method"

Browse Source
This commit is contained in:
Jenkins
2017-06-13 19:28:48 +00:00
committed by Gerrit Code Review
parent a2ad129f72 2a77d130b8
commit bc34e01d29
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
stackalytics/processor/record_processor.py
View File

@@ -237,7 +237,7 @@ class RecordProcessor(object):
def _need_to_fetch_launchpad(self): def _need_to_fetch_launchpad(self):
return CONF.fetching_user_source == 'launchpad' return CONF.fetching_user_source == 'launchpad'
def update_user(self, record): def _update_user(self, record):
email = record.get('author_email') email = record.get('author_email')
user_e = user_processor.load_user( user_e = user_processor.load_user(
self.runtime_storage_inst, email=email) or {} self.runtime_storage_inst, email=email) or {}
@@ -307,7 +307,7 @@ class RecordProcessor(object):
return user return user
def _update_record_and_user(self, record): def _update_record_and_user(self, record):
user = self.update_user(record) user = self._update_user(record)
record['user_id'] = user['user_id'] record['user_id'] = user['user_id']
if user.get('user_name'): if user.get('user_name'):