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

Enable running CI with auto_explain enabled. #259

Closed
emelsimsek opened this issue Sep 28, 2022 · 1 comment
Closed

Enable running CI with auto_explain enabled. #259

emelsimsek opened this issue Sep 28, 2022 · 1 comment

Comments

@emelsimsek
Copy link

In order to make sure Citus works fine with auto_explain, let's enable running CI with auto_explain enabled. Here is an example configuration:

postgresql.conf
shared_preload_libraries = 'citus,pg_stat_statements,auto_explain'
auto_explain.log_min_duration = 0
auto_explain.log_analyze = true
auto_explain.log_buffers = true
auto_explain.log_nested_statements = true

auto_explain comes built-in with postgresql. No need for additional install.

As an important note, those new tests in CI, as they are likely to fail, should not block github merge workflow for PRs at the moment.

@aykut-bozkurt
Copy link
Contributor

aykut-bozkurt commented Oct 14, 2022

After discussion with @emelsimsek, we decided to support that at citus repo for now.
But we also have a plan to run citus tests with any other extensions created with the help of extension tests automation.
You can follow the issue #262 to be aware of the task status.

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

No branches or pull requests

2 participants