From 2a77d130b8f216cfa54710bdd3e1d0e43edc64da Mon Sep 17 00:00:00 2001 From: Ken'ichi Ohmichi Date: Mon, 12 Jun 2017 15:49:22 -0700 Subject: [PATCH] Rename update_user() as an internal method 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 --- stackalytics/processor/record_processor.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/stackalytics/processor/record_processor.py b/stackalytics/processor/record_processor.py index f801cba9f..0a03626bb 100644 --- a/stackalytics/processor/record_processor.py +++ b/stackalytics/processor/record_processor.py @@ -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'):