search

Andrei-Mihailov / ugc_sprint_1

Спринт 8. Репозиторий для командного проекта модуля UGC
0 stars 0 forks source link

Code review #9

Closed BigDeepBlue closed 5 months ago

BigDeepBlue commented 5 months ago

Несколько рекомендаций:

  1. Скобки в таких вот (тут и тут) случаях не нужны.
  2. Вообще, магический метод __init__ придумали для инициализации полей объекта класса. А у вас тут просто указание типа. И IDE начинает ругатся ниже, что Unresolved attribute reference 'clickhouse_connect' for class 'Clickhouse' и mypy говорит, что ugc/etl/config.py:44: error: Item "None" of "Optional[Any]" has no attribute "execute"
  3. Вот в этом месте обратите внимание на https://stackoverflow.com/questions/41686829/why-does-pycharm-warn-about-mutable-default-arguments-how-can-i-work-around-the Присваивать в качестве параметра по умолчанию изменяемый объект - это всегда опасно.

  4. Вот такое еще было задание, где предполагалось сравнение ClickHouse и Vertica (где бы это посмотреть?):

    В результате вашего исследования у вас должны быть:

    • Числа скорости вставки и чтения данных в хранилищах.
    • Схемы хранения и обработки данных в разных хранилищах.
    • Скрипты, которые загружают или генерируют данные в хранилища.
    • Какая-то дополнительная информация о работе с хранилищами, которая поможет принять взвешенное решение.
BigDeepBlue commented 5 months ago

LGTM