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

Delete indexed data when leaving a project #427

Closed
4 tasks
achou11 opened this issue Jan 10, 2024 · 0 comments · Fixed by #469
Closed
4 tasks

Delete indexed data when leaving a project #427

achou11 opened this issue Jan 10, 2024 · 0 comments · Fixed by #469
Assignees
Labels
mvp Requirement for MVP

Comments

@achou11
Copy link
Member

achou11 commented Jan 10, 2024

Description

Follow-up item based on discussions around #410. Implementing it requires updating the multi-core and sqlite indexers first, and then updating the IndexWriter and MapeoProject.leave() to take advantage of the changes.

Tasks

  • Update multi-core-indexer

  • Update mapeo-sqlite-indexer

  • Add a method to IndexWriter to call the relevant method for each sqlite indexer instance

  • Update MapeoProject.leave() implementation to use the new methods. Also have to keep in mind that the order in which they're called is important:

    1. Delete core data (already implemented)
    2. Reset multicore indexer state
    3. Delete data from index tables
@ximenabb ximenabb added the mvp Requirement for MVP label Feb 8, 2024
EvanHahn added a commit that referenced this issue May 20, 2024
When you leave a project, we now delete all indexed data outside of the
`auth` namespace. Fixes [#427].

[#427]: #427

Co-Authored-By: Andrew Chou <andrewchou@fastmail.com>
EvanHahn added a commit that referenced this issue May 20, 2024
When you leave a project, we now delete all indexed data outside of the
`auth` namespace. Fixes [#427].

[#427]: #427

Co-Authored-By: Andrew Chou <andrewchou@fastmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mvp Requirement for MVP
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants