Skip to content

Update PGD limitations - #6940

Merged
julienmarcbrown merged 9 commits into
developfrom
dev/pe/multi-db
Sep 11, 2025
Merged

julienmarcbrown merged 9 commits into
developfrom
dev/pe/multi-db

Conversation

@eatonphil

@eatonphil eatonphil commented Aug 18, 2025 •

Copy link
Copy Markdown
Contributor

BDR-6956

@eatonphil
eatonphil requested a review from a team as a code owner August 18, 2025 15:38
Comment on lines +82 to +90
Because of many immediate risks and limitations, PGD 6 does not
officially support multiple databases on a single PostgreSQL
instance. A future release of PGD 6 may remove these risks and
limitations, but until then **we currently recommend a
one-database-per-instance configuration**. This best practice is also
reflected in our tooling, including the CLI and Connection Manager,
which are designed to work at the instance level.

It's best practice and we recommend that you configure only one database per PGD instance.
These immediate risks and current limitations include:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

My opinion and recommendations:

  • scratch any hints on plans for future from the docs. At best, it makes customers eagerly waiting, then impatient. At worst, it's going to be embarrassing for us and disappointing customers down the road.
  • "does not support" contradicts the recommendation, IMO. Let's make it clearer. I wouldn't want any customer nor our field even attempting to try multiple databases on PGD.
  • Scratch the entire "it's still possible to host up to 10 databases" and limitations section. It's not really possible.

For example, how does the Connection Manager "work at instance level"? What if a node that's currently a write leader (and the Connection Manager therefore routes traffic to "at instance level") isn't even part of another database's PGD cluster?

How about Lag Control? Would that work? I rather think it might be missing from the limitations.

Overall, I think "not supported, but we tell you how to work around it" is just inconsequent and unnecessarily creates a foot gun.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Took another shot at it.

Comment thread product_docs/docs/pgd/6/known_issues.mdx
SELECT count(*) FROM pg_largeobject;
```

### Slow performance for tables without primary keys or unique constraints

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I don't like this, as it's not this simple, for very small tables (common use for nonindexed tables) things can be actually faster without index, and on the other hand, replication of large update/delete query that affected millions of rows will be slow even with indexes, so we need bit more precise language and potentially 2 items for this

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Ok I have edited this section to clarify these different scenarios.

the right order of commits.

- CLI integration assumes one database.
Because of many immediate risks and limitations, we currently

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think this makes sense, however we do allow for up to 10 dbs and sell that sometimes, but maybe at the current stage we can document it like this

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

keeping it as it is for now then I guess

@eatonphil eatonphil changed the title Clarify multiple databases limitation Update known limitations Aug 27, 2025
@eatonphil eatonphil changed the title Update known limitations Update PGD limitations Aug 27, 2025
@eatonphil
eatonphil requested review from PJMODOS and mwanner August 28, 2025 13:21

@irionr irionr left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@julienmarcbrown
julienmarcbrown merged commit aa9d3d3 into develop Sep 11, 2025
7 checks passed
@julienmarcbrown
julienmarcbrown deleted the dev/pe/multi-db branch September 11, 2025 05:59
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.

5 participants