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

fix: remove event_sink schema #123

Closed
wants to merge 2 commits into from

Conversation

saraburns1
Copy link
Contributor

@saraburns1 saraburns1 commented Oct 8, 2024

Materialized views that are not the original data source should not specify a schema so they are created in the 'reporting' schema

@saraburns1 saraburns1 changed the title fix: use event_sink tables fix: remove event_sink schema Oct 8, 2024
@bmtcril bmtcril requested a review from Ian2012 October 8, 2024 20:06
@Ian2012
Copy link
Contributor

Ian2012 commented Oct 8, 2024

Is this what we really want? There are other MVs that are created on the event_sink database too such as most_recent_couse_blocks

@saraburns1
Copy link
Contributor Author

Is this what we really want? There are other MVs that are created on the event_sink database too such as most_recent_couse_blocks

Yeah I'm going to look into it a bit more

@saraburns1 saraburns1 closed this Oct 9, 2024
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.

2 participants