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

Більше демо наповнення #70

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Kobzar09
Copy link

@Kobzar09 Kobzar09 commented Jul 17, 2017

Додав демо наповнення (структура та теми)

Closes #34

@yukoff
Copy link
Contributor

yukoff commented Jul 17, 2017

Не поспішай. По перше, до чого це відноситься? Використовуй GitHub Closing Words, по-друге, якщо це дані ідентичні старій толоці - який в цьому сенс, якщо у #46 робимо процес апдейту бази, а у #7 робиться міграція власне старих даних.

До того ж, я за те, щоб робити дамп виключно mysqldump - наприклад, на Percona Server 5.7.18 через

SET AUTOCOMMIT = 0;
START TRANSACTION;

ловлю помилку про дублікат PK, до того ж кожен CREATE TABLE, ALTER TABLE тощо роблять COMMIT - суть від цього при початковому заливанні бази сумнівна.

@yukoff
Copy link
Contributor

yukoff commented Jul 17, 2017

internal_data/cache/.htaccess 100644 → 100755

Перевіряй, що додаєш в коміт

git status
git add ...
git diff --cached

@Kobzar09
Copy link
Author

@yukoff пан Конфуцій поставив завдання #34. Я його виконав.

@yukoff
Copy link
Contributor

yukoff commented Jul 17, 2017

@Kobzar09

пан Конфуцій поставив завдання #34

Оце інша справа, але це треба писати при створенні PR + все ж використовуй GitHub Closing Words

У бідь якому випадку, прибери атрибут виконання
з internal_data/cache/.htaccess

git update-index --chmod=-x internal_data/cache/.htaccess
git commit --amend

Щоб додавати до комміту тільки змінені файли

git add -u .

@yukoff
Copy link
Contributor

yukoff commented Jul 17, 2017

@Kobzar09 Щодо "закриваючих слів GitHub" - поправив твій опис PR, щоб зрозуміло було, в даному випадку

...

Closes #34

@yukoff yukoff requested review from yukoff and konfuciusu July 17, 2017 18:47
@konfuciusu
Copy link
Contributor

Треба зробити дамп з англійською версією phpmyadmin.

@Kobzar09
Copy link
Author

Чому стандартно стоїть російська мова в phpmyadmin?

@Kobzar09
Copy link
Author

Розберуся, як тут оновлювати файли, додам.

@yukoff
Copy link
Contributor

yukoff commented Jul 18, 2017

@Kobzar09

Чому стандартно стоїть російська мова в phpmyadmin?

Та хто ж його зна... Я його у себе з docker-compose.yml взагалі випиляв, бо як на мене простіше зробити

mysqldump --compact --comments --skip-extended-insert toloka > install/sql/mysql.sql

@konfuciusu До речі, треба б мускульного клієнта додати в контейнер толоки - це спростить підготовку дампу. Ну або для мускульного контейнера додати відповідний volume

Copy link
Contributor

@yukoff yukoff left a comment

Choose a reason for hiding this comment

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

Потрібно оновити згідно коментарів (особливо прибрати зайве з коміту)

@yukoff yukoff added pr and removed pr labels Jul 18, 2017
@konfuciusu
Copy link
Contributor

@Kobzar09

Чому стандартно стоїть російська мова в phpmyadmin?

Він вибирає мову за налаштуваннями браузера. Подивіться на Accept Language.

@yukoff

треба б мускульного клієнта додати в контейнер толоки - це спростить підготовку дампу. Ну або для мускульного контейнера додати відповідний volume

Правильно дамп робити так (взято з library/mariadb):

docker exec mysql sh -c 'exec mysqldump -uroot -proot toloka' > toloka.sql

@Kobzar09
Copy link
Author

@konfuciusu в мене стоїть українська стандартна

@Kobzar09
Copy link
Author

Ще теми додати у всіх розділах?
Чи так достатньо?

@yukoff
Copy link
Contributor

yukoff commented Jul 18, 2017

@konfuciusu Дійсно, я перемудрував =) З --compact дамп буде компактнішим, але він також включає --skip-comments, тож примусово після нього --comments. Я б ще --order-by-primary додав.

@Viach
Copy link

Viach commented Jul 21, 2017

я так розумію, що це вже буде повноцінна демо база і можна буде приступати до створення API ?

@yukoff
Copy link
Contributor

yukoff commented Jul 23, 2017

@Viach Щодо API - я пропоную спочатку специфікацію розробити, не кидатись одразу в код. Плюс щось мені здається, що використовувати повноцінний веб фреймворк (django) лише для створення API - перебір. З того, що я нашвидкоруч подивився - поки що нода в фаворитах. До того ж API запланований вже на наступну ітерацію (v3.1.0), зараз працюємо над v3.0.0

@Viach
Copy link

Viach commented Jul 23, 2017

справа ваша. node.js - не мій профіль.

щодо перебору - так , можливості Django набагато більші.
можна написати 20 рядків pure python (для такого API, як зараз існує - вистачить) - але не бачу поганого в тому, щоб викостиавши Django чи Flask мати можливість легко і природньо потім розширити сервіс, наприклад додавши веб-форму для зручного пошуку по жанрах, фільтри по роках чи розмірах файлів і т.п.
/* Now is better than never. (The Zen of Python) */

@yukoff
Copy link
Contributor

yukoff commented Jul 23, 2017

@Viach Та з рештою - мова/фреймворк, то справа десята ;) Я теж ноду не юзаю постійно. Я ж кажу - почати варто зі специфікації. А там до речі і видно буде краще - що більш підходить, що менш.

@yukoff
Copy link
Contributor

yukoff commented Jul 25, 2017

Review status: 0 of 1 files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.


install/sql/mysql.sql, line 24 at r1 (raw file):

-- База даних: `toloka`
--
CREATE DATABASE IF NOT EXISTS `toloka` DEFAULT CHARACTER SET latin1 COLLATE latin1_swedish_ci;

Перевір налаштування - має бути типово utf8mb4 + utf8mb4_unicode_ci


Comments from Reviewable

@yukoff yukoff changed the title Toloka db Більше демо наповнення Jul 25, 2017
@yukoff yukoff added the db label Jul 25, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants