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.

Sign up for GitHub

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

Jump to bottom

Migration problem in 8.0 - index not migrating correctly #704

Closed
aboodman opened this issue Nov 18, 2021 · 1 comment · Fixed by #708
Closed

Migration problem in 8.0 - index not migrating correctly #704

aboodman opened this issue Nov 18, 2021 · 1 comment · Fixed by #708
Assignees
@arv

Comments

@aboodman
Copy link
Contributor

aboodman commented Nov 18, 2021 •
edited
Loading

See screen cap of stack:

Screen Shot 2021-11-17 at 3 50 33 PM (1)

There is also an Rocicorp internal notion doc with the exported data that causes this problem: https://www.notion.so/11-17-2017-076cfa7d575c48158e6113e73d83546a#58781d070c884da3a83627df4e55afb2

This prevents @m0c in discord from updating his users to 8.0.
@aboodman
Copy link
Contributor Author

@arv can you take a look at this?

arv added a commit to arv/replicache that referenced this issue Nov 18, 2021
@arv
fix: Fix migraion of head
aa76e09 
The migration from v1 to v2 was not updating the head so the migrated
dag was GC'ed and the old dag was being kept.

Fixes rocicorp#704
@arv arv mentioned this issue Nov 18, 2021
Merged
@arv arv closed this as completed in #708 Nov 18, 2021
arv added a commit that referenced this issue Nov 18, 2021
@arv
fix: Fix migraion of head (#708)
8c50cb4 
The migration from v1 to v2 was not updating the head so the migrated
dag was GC'ed and the old dag was being kept.

Fixes #704
arv added a commit that referenced this issue Nov 22, 2021
@arv
fix: Fix migraion of head (#708)
e5addc7 
The migration from v1 to v2 was not updating the head so the migrated
dag was GC'ed and the old dag was being kept.

Fixes #704
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@arv arv

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: Fix migraion of head arv/replicache
2 participants
@arv @aboodman