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

[PGPRO-5340] #44

Closed
wants to merge 1 commit into from
Closed

[PGPRO-5340] #44

wants to merge 1 commit into from

Conversation

Belovchenko
Copy link

Added new GUC variable 'aqo.profile_mem' and hash table in shared memory for storing query hash and execution time values

Tags: aqo

in shared memory for storing query hash and execution time values

Tags: aqo
@danolivo
Copy link
Collaborator

Есть небольшие замечания:

  • Код стоит вынести в отдельные c/h файлы
  • неоднократно видел висячие пробелы и лишние пустые строки
  • если внутри if только один оператор, то обычно фигурные скобки не ставятся
  • hash_search() может генерировать ошибку 'out of memory' - AQO не должен здесь выдавать ошибок, он должен однократно записать сообщение в лог и далее игнорировать такие ошибки - просто не добавлять записей. Здесь нужно использовать конструкцию PG_TRY() - PG_CATCH()

@danolivo danolivo closed this Nov 18, 2021
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.

None yet

2 participants