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

Tile proxy model should leverage published graph #9734

Closed
chiatt opened this issue Jul 6, 2023 · 0 comments · Fixed by #9753
Closed

Tile proxy model should leverage published graph #9734

chiatt opened this issue Jul 6, 2023 · 0 comments · Fixed by #9753
Assignees
@aarongundel
Labels
Subject: Performance

Comments

@chiatt
Copy link
Member

chiatt commented Jul 6, 2023

There are a number of places in the tile model that rely on database queries - particularly to collect node data. Where possible these queries should leverage the serialized graph.
@chiatt chiatt changed the title Tile and Resource proxy models should leverage published graph Tile proxy model should leverage published graph Jul 6, 2023
@chiatt chiatt moved this to 🏗 In progress in afs-pipeline Jul 7, 2023
@aarongundel aarongundel moved this from 🏗 In progress to 👀 In review in afs-pipeline Jul 10, 2023
@chiatt chiatt moved this to 👀 In Review in pipeline Jul 25, 2023
@jacobtylerwalls jacobtylerwalls linked a pull request Sep 18, 2023 that will close this issue
uses serialized graph in tile model #9753
Merged
@github-project-automation github-project-automation bot moved this from 👀 In Review to ✅ Done in pipeline Sep 20, 2023
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in afs-pipeline Sep 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@aarongundel aarongundel

Labels
Subject: Performance
Projects
afs-pipeline
Status: ✅ Done
pipeline
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

uses serialized graph in tile model archesproject/arches
2 participants
@aarongundel @chiatt