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

Migrate dependency graph mv from signoz_index_v2 to signoz_index_v3 #6511

Closed
srikanthccv opened this issue Nov 22, 2024 · 6 comments · Fixed by SigNoz/signoz-otel-collector#461
Closed

Migrate dependency graph mv from signoz_index_v2 to signoz_index_v3 #6511

srikanthccv opened this issue Nov 22, 2024 · 6 comments · Fixed by SigNoz/signoz-otel-collector#461
Assignees
@nityanandagohain
Labels
request-more-info

Comments

@srikanthccv
Copy link
Member

No description provided.

@request-info Request Info
Copy link

request-info bot commented Nov 22, 2024

We would appreciate it if you could provide us with more info about this issue/pr!

@nityanandagohain
Copy link
Member

Any idea where we use the DependencyGraphTables ? I didn't see them being used in query service apart from the TTL API.

@srikanthccv
Copy link
Member Author

The service map is powered by dependency_graph_minutes_v2

@srikanthccv
Copy link
Member Author

The service map won't work for new users.

@nityanandagohain
Copy link
Member

Aah right, @srikanthccv removing the dependency_graph_minutes and just keep dependency_graph_minutes_v2

@srikanthccv
Copy link
Member Author

Yes, the dependency_graph_minutes is old and not used anymore. Just update the query for dependency_graph_minutes_v2_*mv tables.

@nityanandagohain nityanandagohain mentioned this issue Nov 22, 2024
Merged
srikanthccv pushed a commit to SigNoz/signoz-otel-collector that referenced this issue Nov 22, 2024
@nityanandagohain
fix: update dependency graph mvs (#461)
cf7eff9 
Fixes SigNoz/signoz#6511
Fixes SigNoz/signoz#6512

Part of SigNoz/signoz#5713

* Updating the materialized views to use signoz_index_v3
* deleteing the old dependency graph tables
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nityanandagohain nityanandagohain

Labels
request-more-info
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: update dependency graph mvs SigNoz/signoz-otel-collector
2 participants
@srikanthccv @nityanandagohain