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

Update the collisions Readme with the new replicator details #889

Merged
merged 9 commits into from
Jul 18, 2024

Conversation

tahaislam
Copy link
Contributor

@tahaislam tahaislam commented Feb 27, 2024

What this pull request accomplishes:

  • Updates the collisions README to match the new replicator DAG that copies collisions tables from move_staging to collisions

Closes issue #933

@tahaislam tahaislam self-assigned this Feb 27, 2024
@radumas radumas requested a review from mkewins March 1, 2024 19:46
@gabrielwol gabrielwol assigned gabrielwol and unassigned tahaislam Mar 13, 2024
@gabrielwol
Copy link
Collaborator

Need to update with some new DAG details from #902

@chmnata
Copy link
Collaborator

chmnata commented Apr 18, 2024

@gabrielwol Can you update the doc with the new DAG stuff?

@gabrielwol gabrielwol linked an issue Apr 18, 2024 that may be closed by this pull request
2 tasks
@gabrielwol
Copy link
Collaborator

2. The `collisions_replicator.collision_no` materialized view, which creates a unique id (UID) for each collision, is refreshed.
3. The `collisions_replicator.events` and `collisions_replicator.involved` materialized views are refreshed (more on this below).
4. New records, deletions and updates to existing records are tracked in a table called `collisions_replicator.logged_actions`.
### Updating Existing Tables

Copy link
Collaborator

Choose a reason for hiding this comment

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

nice mention of the dependencies function :P

Copy link
Collaborator

Choose a reason for hiding this comment

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

That was Islam actually 🥲

@mkewins
Copy link
Contributor

mkewins commented Jul 18, 2024

@gabrielwol I made a few small edits and changed some links. LGTM. Please merge + close as appropriate!

@gabrielwol gabrielwol merged commit e8f4c65 into master Jul 18, 2024
5 checks passed
@gabrielwol gabrielwol deleted the doc/move_replicators branch July 18, 2024 20:08
@gabrielwol
Copy link
Collaborator

Thanks for the edits Maddy!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update the Collisions Readme with new replicator details
4 participants