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

Table list not cleared when mirror is deleted #1609

Open
goranc opened this issue Apr 12, 2024 · 6 comments
Open

Table list not cleared when mirror is deleted #1609

goranc opened this issue Apr 12, 2024 · 6 comments

Comments

@goranc
Copy link

goranc commented Apr 12, 2024

When we create a mirror, execute and delete it after we finish.
Then we create a new mirror with the same name to repeat processing,
the list of tables in the initial load screen displays new tables and a list of tables from the previous run.

This behavior is intentional or is it not cascade deleted as it should be?

@iskakaushik
Copy link
Contributor

not intentional @goranc , thanks for filing this issue, we will get to it. cc: @serprex

@serprex
Copy link
Member

serprex commented Apr 17, 2024

@iskakaushik this is somewhat intentional, what should the expected behavior be? Is this only about replacing initial load entries?

@iskakaushik
Copy link
Contributor

@serprex my thought here is that when a flow gets deleted / dropped we should be clearing all the metadata associated with it, for example all the tables in peerdb_stats schema that reference this flow_name and also the errors associated with this flow that are stored in flow_errors table.

@serprex
Copy link
Member

serprex commented Apr 17, 2024

Then we need to come up with another way to recreate mirror while persisting history

@iskakaushik
Copy link
Contributor

fair point, maybe we can have an optional flag in the delete request -- persist history, default it to false

@goranc
Copy link
Author

goranc commented May 6, 2024

Maybe it can be just marked as deleted, to not show history records for new runs with the same name,
but with a flag/switch to show history, to see what was run under the same name (intentional history view).

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

No branches or pull requests

3 participants