Pummas / ugc_sprint_1

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

Code review #38

Open BigDeepBlue opened 1 year ago

BigDeepBlue commented 1 year ago

Не забывайте указывать адрес репозитория в README, того репозитория, который появляется у вас в начале спринта.

Отлично разобрались с ELK 👍 Исследование тоже 🔥

  1. В эндпоинтах в ugc/src/api/v1 в исключениях явно указываете коды ответов сервера. Это не критично, но и не очень хорошо. Используйте https://docs.python.org/3/library/http.html#http.HTTPStatus или fastapi.status.
  2. Давайте попробуем поступить с mongodb также как вы это сделали с Kafka.

      @app.on_event('startup')
      async def startup():
          mongo.mongo_client = AsyncIOMotorClient(settings.MONGO_URL, serverSelectionTimeoutMS=5000)
    
      @app.on_event('shutdown')
      async def shutdown():
          mongo.mongo_client.close()

    Будем ловко один раз подключаться при запуске проекта и разрывать соединение при завершении работы.

  3. Можно попробовать вот это сделать одним запросом. https://www.mongodb.com/docs/drivers/node/current/fundamentals/crud/write-operations/upsert/#performing-an-upsert
Pummas commented 1 year ago

1 - Исправлено 2 - Подключение и так происходит 1 раз, в startup проверяем на наличие коннекта, в shutdown добавил закрытие 3 - Решили оставить с двумя запросами