Skip to content

Commit

Permalink
fix neuploader autostop
Browse files Browse the repository at this point in the history
  • Loading branch information
fomars committed Oct 15, 2019
1 parent 3c7cc69 commit 0e5db0b
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions yandextank/plugins/NeUploader/plugin.py
Expand Up @@ -70,12 +70,12 @@ def data_session(self):

def _cleanup(self):
uploader_metainfo = self.map_uploader_tags(self.core.status.get('uploader'))
self.data_session.update_job(uploader_metainfo)
if self.core.status.get('autostop'):
autostop_rps = self.core.status['autostop'].get('rps', 0)
autostop_reason = self.core.status['autostop'].get('reason', '')
autostop_rps = self.core.status.get('autostop', {}).get('rps', 0)

This comment has been minimized.

Copy link
@szypulka

szypulka Oct 16, 2019

Contributor

а зачем здесь дефолтный {}? Если в статусе нет автостопа, эта строчка выполняться не будет -- там if чуть выше это условие проверяет

This comment has been minimized.

Copy link
@fomars

fomars Dec 12, 2019

Author Contributor

просто это универсальная безопасная практика так писать, когда стоит два get() подряд - если контекст изменится, то в этом месте ничего не сломается. Чем меньше код зависит от контекста, тем меньше потом переписывать и тем больше можно переиспользовать

This comment has been minimized.

Copy link
@szypulka

szypulka Dec 12, 2019

Contributor

Это, мне кажется, сомнительный аргумент -- первый гет полностью дублирует условие входа. Если править, то уж тогда 73 строчку нужно удалять, а 74 и 75 оставлять.
Но вообще это вопрос исключительно вкуса, писать ли в одну строку или с if.

autostop_reason = self.core.status.get('autostop', {}).get('reason', '')
self.log.warning('Autostop: %s %s', autostop_rps, autostop_reason)
self.data_session.update_job({'autostop_rps': autostop_rps, 'autostop_reason': autostop_reason})
uploader_metainfo.update({'autostop_rps': autostop_rps, 'autostop_reason': autostop_reason})
self.data_session.update_job(uploader_metainfo)
self.data_session.close(test_end=self.core.status.get('generator', {}).get('test_end', 0) * 10**6)

def is_test_finished(self):
Expand Down

0 comments on commit 0e5db0b

Please sign in to comment.