komoart / kafka_clickhouse

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

Code review #8

Open BigDeepBlue opened 1 year ago

BigDeepBlue commented 1 year ago

Просто к сведению: использование Engine = Kafka упрощает работу, но лишает нас возможности промежуточного преобразования данных, как это было бы возможно в ETL.

  1. Не нашел реализацию задания:

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

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

  2. Любое API не сможет долго существовать без тестов. Нужно подготовить инфраструктуру для юнит тестов и написать пару.
  3. В ugc_api/svc зачастую забываете указывать типы принимаемых параметров и возвращаемых значений.
  4. Посмотрите очень подробный и интересный пример реализации авторизации в FastAPI https://testdriven.io/blog/fastapi-jwt-auth/ , самое интересное в разделе Securing Routes. Вы неавторизованного пользователя проверяете на уровне сервисов приложения. А в предложенном примере все происходит на уровне эндпоинтов и лишние запросы сразу отсекаются.
BigDeepBlue commented 1 year ago

LGTM