artwist-polyakov / Auth_sprint_1

Спринт 6
3 stars 0 forks source link

Code review #213

Closed BigDeepBlue closed 7 months ago

BigDeepBlue commented 7 months ago
  1. Подскажите где искать API для CRUD лайков, рецензий и закладок, который предполагалось делать с использованием mongoDB? Я не нашел 🤨 В вашем исследовании победил ElasticSearch, и вы вероятно реализовали на ней. Если же еще нет, то предлагаю все же использовать монгу. Не смотря на похожесть у монги и эластика, все же, разные задачи. https://logz.io/blog/elasticsearch-vs-mongodb/
  2. Отлично разобрались с ELK. Вот еще небольшой пример, там наверняка что-то пригодится https://github.com/RomanAVolodin/FastAPI-MongoDb-ELK. Там я добавляю в логи приложения дополнительные поля, которые парсятся и в ELK по ним можно фильтровать и т.д.
artwist-polyakov commented 7 months ago

@BigDeepBlue мы не считали в программе курса задачу на дополнение апи методами хранения лайков рецензий и закладок — сейчас добьём

artwist-polyakov commented 7 months ago

@BigDeepBlue прикрутили ручки апи для работы с оценками есть 2 важных момента.

  1. Так как команда из 2 человек, то мы не переливаем данные из базы фильмов — просто не сможем успеть это
  2. Если ставится оценка фильму — то просто создаётся фильм, как будто такой существует
  3. Если надо поставить оценку рецензии, то надо в начале оставить саму рецензию, рецензия не создаётся автоматически как фильм.
  4. Каскадного удаления лайков и рецензий при удалении фильмов делать не стали — решается одной строкой, но кажется что если фильм появится в кинотеатре снова иметь доступ к его лайкам и рецензиям было бы хорошо.