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

Добавлена команда загрузки информационной базы из файла выгрузки (dt) - restore #488

Merged
merged 5 commits into from
Mar 25, 2022

Conversation

realMaxA
Copy link
Contributor

@realMaxA realMaxA commented Feb 18, 2022

closes #487

@artbear
Copy link
Collaborator

artbear commented Feb 19, 2022

@realMaxA а чем не подходят 2 существующих варианта загрузки - init-dev и update-dev ?

@realMaxA
Copy link
Contributor Author

realMaxA commented Feb 21, 2022

@artbear

@realMaxA а чем не подходят 2 существующих варианта загрузки - init-dev и update-dev ?

Наверное тем, что не ожидалось там найти существующий функционал.
Искал соответствующую функцию рядом с командой dump

По большому счёту восстановление базы из dt не обязательно относится к dev.

@artbear
Copy link
Collaborator

artbear commented Mar 5, 2022

Закрываем ПР, т.к. функционал уже реализован? @realMaxA

@realMaxA
Copy link
Contributor Author

realMaxA commented Mar 6, 2022

@artbear

Закрываем ПР, т.к. функционал уже реализован?

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

Мои аргументы такие:

  1. Восстановление базы из dt не обязательно связано с dev-контуром, а существующие команды vrunner как бы намекают на это (init-dev, update-dev)
  2. При поиске необходимой функциональности в help по команде ожидаешь найти реализацию команды конфигуратора restore и совершенно не ожидаешь, что она находится где-то в других командах. Особенно, если твои задачи не связаны с dev-контуром (в команды *-dev даже не думаешь заглядывать, а в кратком описании этих команд нет даже намёка на возможность разворачивания базы из dt)

@artbear artbear added this to the 1.11.0 milestone Mar 25, 2022
@artbear artbear changed the title #487. Добавлена команда загрузки информационной базы из файла выгрузки (dt) Добавлена команда загрузки информационной базы из файла выгрузки (dt) - restore Mar 25, 2022
@artbear
Copy link
Collaborator

artbear commented Mar 25, 2022

@realMaxA да, согласен с твоим ПР и доводами в его пользу. Сливаю.

Большое спасибо!

@artbear artbear merged commit cf13fba into vanessa-opensource:develop Mar 25, 2022
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.

Команда Загрузки информационной базы из файла - restore
2 participants