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

Проблема с stats_temp_directory #54

Closed
sgrinko opened this issue Aug 17, 2020 · 7 comments
Closed

Проблема с stats_temp_directory #54

sgrinko opened this issue Aug 17, 2020 · 7 comments
Assignees
Labels
enhancement Improvement of the current functionality

Comments

@sgrinko
Copy link

sgrinko commented Aug 17, 2020

Добрый день!

Спасибо большое за вашу работу!

Параметр stats_temp_directory в postgres позволяет указать где хранить данные сборщика статистики и этот каталог можно делать прямо в ОЗУ (tmpfs) чтобы улучшить ввод вывод для этих операций. Прекрасно, что в данной реализации скриптов учтен этот важный момент, однако пару моментов не учтены.

  1. нет возможности указать размер для такой директории в ОЗУ
  2. нет возможности сменить имя для этой директории
  3. нет возможности просто отказаться от такой возможности и оставить default вариант - каталог на диске как обычно.

в качестве частичного варианта реализации пункта 3 я использовал --skip-tags "pgsql_stats_tmp", но при этом пришлось вручную комментировать соответствующий параметр в roles/patroni/templates/patroni.yml.j2:
# stats_temp_directory: /var/lib/pgsql_stats_tmp

Предлагаю внести соответствующие параметры для этого в vars/main.yml
stats_temp_directory_install
stats_temp_directory_path
stats_temp_directory_size

@vitabaks
Copy link
Owner

  1. Выделяется до 1gb, большее значение не понадобится. По факту, используется не весь выделенный объём, а столько сколько требуется на данный момент. Проверьте с помощью df - h

  2. С какой целью? (будет добавлена возможность указать другой путь ).

  3. С какой целью? Прироста производительности это не даст. Стоит ли экономить пару мегабайт памяти это вопрос. (будет реализована возможность отключить создание в tmpfs).

@vitabaks vitabaks self-assigned this Aug 17, 2020
@vitabaks vitabaks added the enhancement Improvement of the current functionality label Aug 17, 2020
@sgrinko
Copy link
Author

sgrinko commented Aug 17, 2020

Как я могу к вам обращаться ? Ко мне можно просто по имени - Сергей

  1. сейчас выделяется 1 ГБ, сколько используется могу посмотреть конечно :)
    дело в том, что не на всех серверах достаточно ОЗУ, чтобы выделить целый Гигабайт для этого. У меня есть сервера у которых всего то 2 ГБ ОЗУ :)
  2. другой путь может понадобиться если к примеру устанавливается постгрес на сервер, где уже стоит другой постгрес. В целях тестирования такое может быть легко.
  3. как я уже писал выше, в целях выделения ОЗУ для других, более насущных, целей :)

@vitabaks
Copy link
Owner

Вы меня не поняли. По факту, не потребляется гигабайт.

@sgrinko
Copy link
Author

sgrinko commented Aug 17, 2020

Я вас понял прекрасно :)
Но я к примеру на очень многих серверах выделять для этого в ОЗУ часть памяти не могу.
Думаю, что иметь возможность делать эти действия гибче будет хорошо для всех.

@vitabaks
Copy link
Owner

Будет реализовано.

vitabaks added a commit that referenced this issue Aug 17, 2020
postgresql_stats_temp_directory_path
postgresql_stats_temp_directory_size

#54
@vitabaks
Copy link
Owner

vitabaks commented Aug 17, 2020

Готово (3be08fc).

Переменные postgresql_stats_temp_directory_path и postgresql_stats_temp_directory_size
в vars/Debian.yml либо vars/RedHat.yml

@sgrinko
Copy link
Author

sgrinko commented Aug 17, 2020

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Improvement of the current functionality
Projects
None yet
Development

No branches or pull requests

2 participants