Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sprint2 #22

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Conversation

EgorovEvgen1991
Copy link

Имеются вопросы к ревьюверу. Просьба посмотреть.

@@ -0,0 +1,3 @@
1) в ходе реализации заметил что json выводит шарды как реплики. ошибка вывода логики в приложении или это особенность MongoDB? (см скрин 1)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

не совсем понял про какое поле речь, но тут, скорее всего, вопрос терминологии. Репликасет в mongodb состоит из нескольких реплик, это мастер-реплика и одна и более слейв-реплик.
На самом деле там может больше ролей, подробнее можно почитать в документации
https://www.mongodb.com/docs/manual/replication/#replication-in-mongodb

A replica set is a group of mongod instances that maintain the same data set. A replica set contains several data bearing nodes and optionally one arbiter node. Of the data bearing nodes, one and only one member is deemed the primary node, while the other nodes are deemed secondary nodes.

@@ -0,0 +1,3 @@
1) в ходе реализации заметил что json выводит шарды как реплики. ошибка вывода логики в приложении или это особенность MongoDB? (см скрин 1)
2) При проверки задания 4 (запрос списка пользователей)в основное приложение выкидывается ошибка. не пойму в чем дело. я что-то делаю неправильно или косяк в основном приложении?

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

такая проблема бывает из за кластерии редиса, на сколько я знаю, по заданию не обязательно поднимать кластер, достаточно одного экземпляра редиса.
Поэтому тут можно доработать (и поправить конфиг) и ошибка скорее всего уйдет

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants