x/stackalytics
Code Issues Proposed changes

Rename update_user() as an internal method

Browse Source 
update_user() was used in record_processor module only, so that should
be considered as an internal method. This patch renames it.

Change-Id: I11ec5baab9e89270a593c2aa8eb32318135b28b9
This commit is contained in:
Ken'ichi Ohmichi
2017-06-12 15:49:22 -07:00
parent fa899f0e23
commit 2a77d130b8
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):
return CONF.fetching_user_source == 'launchpad'
def update_user(self, record):
def _update_user(self, record):
email = record.get('author_email')
user_e = user_processor.load_user(
self.runtime_storage_inst, email=email) or {}
@@ -307,7 +307,7 @@ class RecordProcessor(object):
return user
def _update_record_and_user(self, record):
user = self.update_user(record)
user = self._update_user(record)
record['user_id'] = user['user_id']
if user.get('user_name'):