Skip to content

Доработка команды "ALL" - реализован функционал использования готовой… #255

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

Merged
merged 3 commits into from
May 17, 2021

Conversation

Dach-Coin
Copy link
Contributor

… базы, подключаемой по строке "ibconnection" (решение проблемы наличия аппаратных лицензий для временных файловых баз)

реализация #

сделанные изменения:

  • фактически всего лишь организовал передачу 3 параметров (строка соединения, пользователь, пароль) до метода "Установить контекст", который уже успешно работает для команды "SYNC", после чего заработало и для команды "ALL"
  • протестировал, для вот такой строки вызова: "gitsync --db-user Администратор --db-pwd 123456 -v --v8version 8.3.18.1289 --ibconnection /S192.168.0.102:2541\TempBase1C --tempdir ./.gitsync_tmp all ./tools/JSON/gitsync_conf.json"
    успешно использует пакетный запуск конфигуратора базы "192.168.0.102:2541\TempBase1C"
  • если использовать вместо готовой серверной базы готовую файловую, например: "gitsync -v --v8version 8.3.18.1289 --ibconnection /FD:\SQL\1C_db\Empty_base --tempdir ./.gitsync_tmp all ./tools/JSON/gitsync_conf.json" - тоже успешно работает, используется заданная готовая файловая база

@oscript-library/Collaborators - пожалуйста, проверьте изменения и прокомментируйте их

Dach-Coin added 3 commits May 10, 2021 12:36
… базы, подключемой по строке ibconnection (решение проблемы наличия аппаратных лицензий для временных файловых баз)
@Dach-Coin
Copy link
Contributor Author

Протестировано в том числе на виртуалке Linux, на которую не проброшены лицензии

@khorevaa
Copy link
Member

khorevaa commented May 12, 2021

Мне думается допущена логическая ошибка..
При работе с данным ПР получится, что для синхронизации по списку хранилищ будет всегда использоваться одна и та же информационная база.. я думаю это не верно, или частично верно..

Правильнее сделать чтобы параметры подключения указывались для конкретной настройки синхронизации в команде ALL

@nixel2007 @artbear @EvilBeaver - коллеги. призываю к обсуждению данного вопроса.

@Dach-Coin
Copy link
Contributor Author

Мне думается допущена логическая ошибка..
При работе с данным ПР получится, что для синхронизации по списку хранилищ будет всегда использоваться одна и та же информационная база.. я думаю это не верно, или частично верно..

Правильнее сделать чтобы параметры подключения указывались для конкретной настройки синхронизации в команде ALL

@nixel2007 @artbear @EvilBeaver - коллеги. призываю к обсуждению данного вопроса.

Цель данного PR - для команды ALL избавиться от использования файловой базы. Сейчас при пакетной синхронизации тоже используется одна и та же файловая база одного и того же "контекста". Научить gitsync для каждого репо из настройки использовать свою готовую серверную базу - цели не было. Серверная база - просто буфер, какая там конфа лежит в тот или иной момент времени - не суть так важно, она же нужна просто чтобы конфу на файлы разложить и при этом задействовать серверную лицензию, а не аппаратную (которой например нет на ноде)

@nixel2007
Copy link
Member

@khorevaa в принципе последний комментарий @Dach-Coin кажется вполне логичным.

@khorevaa
Copy link
Member

Тогда у меня нет возражений

@khorevaa khorevaa merged commit 651996a into oscript-library:develop May 17, 2021
@khorevaa
Copy link
Member

@Dach-Coin Благодарю за PR

@nixel2007
Copy link
Member

@khorevaa релизнешь? И плагины заодно, там были новые доработки и сами плагины

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.

3 participants