Ревью в проекте

Question

Closed this issue 2 years ago · 1 comments

Проведите ревью 2 заданий своего коллеги по команде. Можно брать задания из спринта по ETL, и первого спринта из модуля "Сервис Async API".

Оценка: 5

Ссылка на PR: pass

Answer 1 · 2022-10-05T16:48:30.000Z

Ревью 2 заданий по ETL

Информационный лог с уровнем warning

        def wrapper(*args, **kwargs) -> Callable:
            time.sleep(secs)
            return func(*args, **kwargs)

wrapper возвращает не Callable объект, а результат выполнения func(*args, **kwargs)
Декоратор засыпает до выполнения функции, логичнее засыпать после выполнения
wrapper лучше оборачивать в декоратор functools.wraps чтобы сохранять атрибуты оригинальной функции https://docs.python.org/3/library/functools.html#functools.wraps

"""Декоратор для поаторного соединения с ElasticSearch"""
                   ^

wrapper лучше оборачивать в декоратор functools.wraps чтобы сохранять атрибуты оригинальной функции https://docs.python.org/3/library/functools.html#functools.wraps
Заявлено: Декоратор для поаторного соединения с ElasticSearch. По факту возвращает результат выполнения функции extractor, а если ES свалится, то выполнит только логирование и зарайзит ошибку - реконнекта выполнено не будет.
Имя переменной extractor не соответствует тому как она используется в декораторе
Информациооное сообщение логирования со статусом warning

Те же ошибки что и в es_reconnect.py

Придирка: библиотека pydantic подключена, датаклассы легко можнозаменить и использовать валидацию от пидантика:

from dataclasses import dataclass

Заменив на:

from pydantic.dataclasses import dataclass

@dataclass(frozen=True, slots=True)

def connected(self) -> bool | None:
    return self._connection and self._connection.closed == 0

Если self._connection заменить на

self._connection is not None

Возвращаться будет только bool и проверка в if будет более правильной

Название функции connect не соответствует заявленному функционалу: "Функция пересоздает соединение с БД"
Придирка: проставлены не все аннотации типов

Закомментирован неиспользуемый код