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

Extend intelmq_psql_initdb #2404

Merged
merged 10 commits into from
Sep 26, 2023
Merged

Extend intelmq_psql_initdb #2404

merged 10 commits into from
Sep 26, 2023

Conversation

kamil-certat
Copy link
Contributor

Added creating view for separating raws, optional partitioning key,
custom harmonization file path and if not exists/or replace clauses.

Those changes improves the capabilities of the database
schema creation. It's now easier to switch to the TimescaleDB
or use partitioning in PostgreSQL. Creating scripts can be
generated from the harmonisation, as previously just the most
basic version. If requested, creation commands get IF NOT EXISTS
or OR REPLACE directives to allow re-runing script.

Those changes improves the capabilities of the database
schema creation. It's now easier to switch to the TimescaleDB
or use partitioning in PostgreSQL. Creating scripts can be
generated from the harmonisation, as previously just the most
basic version. If requested, creation commands get IF NOT EXISTS
or OR REPLACE directives to allow re-runing script.
@kamil-certat kamil-certat requested a review from sebix August 31, 2023 12:19
@kamil-certat kamil-certat added support component: contrib feature Indicates new feature requests or new features and removed support labels Aug 31, 2023
Copy link
Member

@sebix sebix left a comment

Choose a reason for hiding this comment

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

Great to see that my design is still useful :)

Is the file contrib/eventdb/separate-raws-table.sql obsoleted?

https://intelmq.readthedocs.io/en/develop/user/eventdb.html#separating-raw-values-in-postgresql-using-view-and-trigger needs an update for the new method.

@kamil-certat
Copy link
Contributor Author

Thanks for catching lack of docs, let's see now

Copy link
Member

@sebix sebix left a comment

Choose a reason for hiding this comment

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

Only wording suggestions. LGTM

@kamil-certat
Copy link
Contributor Author

@sebix Would you find time to merge this PR?

@sebix sebix added this to the 3.2.2 milestone Sep 26, 2023
@sebix sebix merged commit 8c7bef1 into certtools:develop Sep 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: contrib feature Indicates new feature requests or new features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants