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

feat!: Include versions in IDB name #741

Merged
merged 1 commit into from Dec 2, 2021
Merged

feat!: Include versions in IDB name #741

merged 1 commit into from Dec 2, 2021

Conversation

arv
Copy link
Contributor

@arv arv commented Dec 1, 2021

For Simplified Dueling Dags we need to ensure that different tabs
running different versions of Replicache do not interact with IDB data
it does not know how to read/write.

To achieve this the name if the IDB database now contains the
REPLICACHE_FORMAT_VERSION (which is currently at 3).

The IDB name also contains the schemaVersion som if the schema changes
a fresh IDB database is used. The motivation is the same. Multiple tabs
with different schemaVersions should not interact with the same IDB
database.

BREAKING CHANGE

Towards #671

For Simplified Dueling Dags we need to ensure that different tabs
running different versions of Replicache do not interact with IDB data
it does not know how to read/write.

To achieve this the name if the IDB database now contains the
`REPLICACHE_FORMAT_VERSION` (which is currently at `3`).

The IDB name also contains the `schemaVersion` som if the schema changes
a fresh IDB database is used. The motivation is the same. Multiple tabs
with different schemaVersions should not interact with the same IDB
database.

BREAKING CHANGE

Towards rocicorp#671
@arv arv requested review from aboodman and grgbkr December 1, 2021 23:46
@vercel
Copy link

vercel bot commented Dec 1, 2021

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/rocicorp/replicache/Fyukp7EdZq3J72biaBqXg3B8DE5R
✅ Preview: https://replicache-git-fork-arv-idb-name-rocicorp.vercel.app

@vercel vercel bot temporarily deployed to Preview December 1, 2021 23:46 Inactive
@arv arv mentioned this pull request Dec 1, 2021
31 tasks
@grgbkr
Copy link
Contributor

grgbkr commented Dec 2, 2021

LGTM

Copy link
Contributor

@grgbkr grgbkr left a comment

Choose a reason for hiding this comment

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

LGTM

@arv arv merged commit 5aa948f into rocicorp:main Dec 2, 2021
@arv arv deleted the idb-name branch December 2, 2021 19:10
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.

None yet

3 participants