-
Notifications
You must be signed in to change notification settings - Fork 673
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
chore: make D1's experimental backend the default #3668
Conversation
🦋 Changeset detectedLatest commit: 21deb80 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
@@ -59,8 +54,8 @@ export const Handler = withConfig<HandlerOptions>( | |||
}, | |||
body: JSON.stringify({ | |||
name, | |||
experimental: true, |
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.
We can remove this once D1's API defaults to the new backend.
Codecov Report
@@ Coverage Diff @@
## main #3668 +/- ##
==========================================
+ Coverage 75.22% 75.29% +0.06%
==========================================
Files 190 190
Lines 11173 11170 -3
Branches 2948 2945 -3
==========================================
+ Hits 8405 8410 +5
+ Misses 2768 2760 -8
|
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.
LGTM. Let's go!
Created your database using D1's new storage backend. The new | ||
storage backend is not yet recommended for production workloads, but | ||
backs up your data via point-in-time restore. |
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.
Should this message be updated?
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.
Eh we can update that once we go from alpha -> beta, D1 is still not recommended for production workloads until then.
* chore: make D1's experimental backend the default * fix: actually remove the experimental backend flag
Fixes #3667
What this PR solves / how to test:
wrangler d1 create <dbname>
wrangler d1 info <dbname>
Associated docs issue(s)/PR(s):
Author has included the following, where applicable:
[ ] TestsReviewer is to perform the following, as applicable: