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

Mark group-commit.mdx with warning #6264

Merged
merged 5 commits into from
Nov 20, 2024
Merged

Mark group-commit.mdx with warning #6264

merged 5 commits into from
Nov 20, 2024

Conversation

djw-m
Copy link
Contributor

@djw-m djw-m commented Nov 20, 2024

What Changed?

Follow up to #6262

@djw-m djw-m requested a review from a team as a code owner November 20, 2024 13:04
@djw-m djw-m requested review from PJMODOS and removed request for a team November 20, 2024 13:04
Copy link
Contributor

@gvasquezvargas gvasquezvargas left a comment

Choose a reason for hiding this comment

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

LGTM

@@ -15,6 +15,10 @@ node to successfully confirm a transaction at COMMIT time. Confirmation can be s
at a number of points in the transaction processing but defaults to "visible" when
the transaction has been flushed to disk and is visible to all other transactions.

!!! Warning

Choose a reason for hiding this comment

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

suggest re-wording since this was released and was out there for long ....
Group commit is currently offered as an experimental feature intended for preview and evaluation purposes. While it provides valuable capabilities, it has known limitations and challenges that make it unsuitable for production environments. We recommend that customers avoid using this feature in production scenarios until these limitations are addressed in future releases.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

@djw-m djw-m added the deploy Add this label to a PR and it will automatically be deployed to netlify label Nov 20, 2024
@djw-m djw-m merged commit a2d7b9e into develop Nov 20, 2024
2 checks passed
@djw-m djw-m deleted the docs/pgd/fix/50update branch November 20, 2024 17:32
Copy link
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deploy Add this label to a PR and it will automatically be deployed to netlify
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants