-
Notifications
You must be signed in to change notification settings - Fork 100
fix(cloudflare): handle large d1 migrations in local dev #1246
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
Conversation
commit: |
🚀 Website Preview DeployedYour website preview is ready! Preview URL: https://a8b8a270-alchemy-website.alchemy-run.workers.dev This preview was built from commit 4eeff51 🤖 This comment will be updated automatically when you push new commits to this PR. |
| // split large migrations to prevent D1_ERROR: statement too long: SQLITE_TOOBIG | ||
| await session.batch( | ||
| migration.sql | ||
| .split(";") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure if splitting by semicolon is a good idea. Drizzle splits by statement breakpoint (https://github.com/drizzle-team/drizzle-orm/blob/c445637df39366bcf47b12601896ce851771c1c2/drizzle-orm/src/migrator.ts#L44)
|
What's the update here? |
No description provided.