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

Storage manager encryption #178

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

Storage manager encryption #178

wants to merge 2 commits into from

Conversation

dutow
Copy link
Collaborator

@dutow dutow commented Apr 15, 2024

This PR currently contains two commits:

  1. makes the current code compile on PG17 without breaking anything on PG16, but not trying to make the extension work with PG17.
  2. adds an SMGR based AES-CBC encryption, which works with the patched PG17 development branch

Copy link

github-actions bot commented Apr 15, 2024

Performance test results:
Normal queries: 9120
TDE queries: 8450
Percentage: 92%

Because the current smgr patchset is based on the PG17 branch, smgr
development needs to be based on that. But our current copied PG16
code doesn't compile with PG17.

This commit doesn't try to make the pg_tde AM work with PG17, only
"comments out" with conditional commands parts that do not compile.

PG16 builds work as before, without any changes.

PG17 at least compiles with missing code, probably doesn't really work.
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

1 participant