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

Fix pg_dump error on powa_module_config #83

Merged
merged 1 commit into from
Dec 18, 2024
Merged

Conversation

rjuju
Copy link
Member

@rjuju rjuju commented Dec 17, 2024

powa_modules are used for cluster-wide pg_stat datasources and do not support custom datasources.

powa_modules and powa_module_functions got it right and don't have the usual "added_manually" column used to distinguish custom rows, but powa_module_functions had a wrong reference to this column. Also, powa_module_config was incorrectly created with an added_manually column, and dump was configure using it but the extension install script was incorrectly marking some of the init-time data to be custom.

To fix, get rid of any mention of that column in either the table structure of the dump config.

Thanks to github user guruguruguru for the report.

powa_modules are used for cluster-wide pg_stat datasources and do not support
custom datasources.

powa_modules and powa_module_functions got it right and don't have the usual
"added_manually" column used to distinguish custom rows, but
powa_module_functions had a wrong reference to this column.  Also,
powa_module_config was incorrectly created with an added_manually column, and
dump was configure using it but the extension install script was incorrectly
marking some of the init-time data to be custom.

To fix, get rid of any mention of that column in either the table structure of
the dump config.

Thanks to github user guruguruguru for the report.
@rjuju rjuju force-pushed the fix_pmf_added_manually branch from 65b6d4b to 8a88d9c Compare December 17, 2024 12:39
@rjuju rjuju merged commit 51067ad into master Dec 18, 2024
9 checks passed
@rjuju rjuju deleted the fix_pmf_added_manually branch December 18, 2024 16:10
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.

1 participant