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

PG-1320 Updated considerations in installation #406

Merged
merged 1 commit into from
Feb 4, 2025

Conversation

nastena1606
Copy link
Collaborator

Added KMIP server to the list of options

Updated formatting for using the count(*) in docs

modified: documentation/docs/decrypt.md
modified: documentation/docs/index.md
modified: documentation/docs/install.md
modified: documentation/docs/setup.md
modified: documentation/docs/test.md

PG-0

Description

Links

@nastena1606 nastena1606 requested review from dutow and dAdAbird January 31, 2025 13:30
@nastena1606 nastena1606 changed the title PG-1320 Updatd considerations in installation PG-1320 Updated considerations in installation Jan 31, 2025
@@ -50,7 +48,7 @@ The `pg_tde` extension comes in two distinct versions with specific access metho

### Which version to chose?

The answer is pretty straightforward: for data sets where indexing is not mandatory or index encryption is not required, use the community version and the `tde_heap_basic` access method. Check the [upstream documentation :octicons-link-external-16:](https://github.com/percona/pg_tde/blob/main/README.md) how to get started.
Use the community version and the `tde_heap_basic` access method for data sets where indexing is not mandatory or index encryption is not required. Check the [upstream documentation :octicons-link-external-16:](https://github.com/percona/pg_tde/blob/main/README.md) how to get started.
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think we can modify this to don't use the basic, as we are deprecating it.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Sure, I'll address this in a separate PR. Otherwise after the merge we will already reveal this info ahead of time.

documentation/docs/install.md Outdated Show resolved Hide resolved
Added KMIP server to the list of options

Updated formatting for using the count(*) in docs

modified:   documentation/docs/decrypt.md
	modified:   documentation/docs/index.md
	modified:   documentation/docs/install.md
	modified:   documentation/docs/setup.md
	modified:   documentation/docs/test.md
@nastena1606 nastena1606 force-pushed the PG-1320-Update-install-add-KMIP branch from d175d8a to f7f21f5 Compare February 4, 2025 11:07
@nastena1606 nastena1606 requested a review from dutow February 4, 2025 11:33
@nastena1606 nastena1606 merged commit 8d0635d into main Feb 4, 2025
9 of 11 checks passed
@nastena1606 nastena1606 deleted the PG-1320-Update-install-add-KMIP branch February 4, 2025 13:30
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.

2 participants