Skip to content

Conversation

@thomasgauvin
Copy link
Contributor

No description provided.

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Oct 10, 2024

Deploying cloudflare-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 6cca59c
Status: ✅  Deploy successful!
Preview URL: https://791d0b35.cloudflare-docs-7ou.pages.dev
Branch Preview URL: https://thomasgauvin-patch-4.cloudflare-docs-7ou.pages.dev

View logs

@Oxyjun
Copy link
Contributor

Oxyjun commented Oct 10, 2024

Hey @vy-ton , we wanted to run this by you - do you think it's worth surfacing any limits of D1 on this page as a factor which helps user decide between using D1 vs external database + Hyperdrive? Or would you rather not draw explicit attention to that?

@Oxyjun Oxyjun self-assigned this Oct 10, 2024
@github-actions
Copy link
Contributor

@vy-ton
Copy link
Contributor

vy-ton commented Oct 10, 2024

Hey @vy-ton , we wanted to run this by you - do you think it's worth surfacing any limits of D1 on this page as a factor which helps user decide between using D1 vs external database + Hyperdrive? Or would you rather not draw explicit attention to that?

should note D1's 10GB database limit and the preferred scale out architecture with many D1 databases

Copy link
Contributor

@vy-ton vy-ton left a comment

Choose a reason for hiding this comment

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

@Oxyjun approved with a few modifications

@thomasgauvin
Copy link
Contributor Author

Hey @vy-ton , we wanted to run this by you - do you think it's worth surfacing any limits of D1 on this page as a factor which helps user decide between using D1 vs external database + Hyperdrive? Or would you rather not draw explicit attention to that?

should note D1's 10GB database limit and the preferred scale out architecture with many D1 databases

I don't think we should note this here. In this section, we compare D1 and Hyperdrive, and have an answer for when to use D1, when to use Hyperdrive.

@Oxyjun Oxyjun enabled auto-merge (squash) October 18, 2024 13:34
@Oxyjun Oxyjun merged commit 11d1e3c into production Oct 18, 2024
13 checks passed
@Oxyjun Oxyjun deleted the thomasgauvin-patch-4 branch October 18, 2024 13:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

product:workers Related to Workers product size/s

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants