Skip to content

Conversation

@randolf
Copy link

@randolf randolf commented Jan 19, 2024

I'm contributing this PostgreSQL database schema as a foundation for adding support for PostgreSQL in the future.

See also: #246

Adding a PostgreSQL database schema to serve as a foundation for adding support for PostgreSQL.
@sparked435
Copy link

LGTM.

Loads perfectly on one of my non-prod instances (Postgres 14.9 w/ Timescale.) Comments look great. Indexes look kind of aggressive but shouldn't be a problem.

@randolf
Copy link
Author

randolf commented Feb 21, 2024

LGTM.

Loads perfectly on one of my non-prod instances (Postgres 14.9 w/ Timescale.) Comments look great. Indexes look kind of aggressive but shouldn't be a problem.

Thanks! Do you happen to know if the organizers of this project are alright? It seems that there are a lot of Pull Requests that need to be addressed.

@sparked435
Copy link

Do you happen to know if the organizers of this project are alright?

Sorry, I'm just some random guy that stumbled over this wondering how to get my mail server to send reports.

Correction to the "disp" column type.
Copy link
Author

@randolf randolf left a comment

Choose a reason for hiding this comment

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

This is a correction of the "disp" column type in the "messages" table.

@thegushi
Copy link
Collaborator

thegushi commented May 4, 2025

So, couple of quick things. Opendmarc does dev work on -develop and MASTER always matches the latest release (this is just a historical thing).

If you're using postgres, have you also modified the reporting scripts to put things in to the database (but not shared them?) I just committed a recent change that adds support for MariaDB (which is functionally mostly the same as mysql but does some warnings). I'd be happy to have the postgres script you're using as well, so we can get this into the commit as well.

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.

3 participants