-
Notifications
You must be signed in to change notification settings - Fork 5
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
Streamline graph DB queries #311
Comments
It might also be worth looking at whether some queries should be managed transactions vs the new |
|
I think we made the decision to table this one for a while until some of the data models are more solidified -- the changes that I have in mind would be quite brittle to schema updates. |
@jsstevenson I plan on updating the normalizers/ga4gh.vrs and then pulling in cat-vrs/va-spec python model changes. |
This issue is stale because it has been open 180 days with no activity. This issue will be closed if no further activity occurs in 14 days. |
This issue was closed because it has been stalled for 14 days with no activity. |
It's been true for a while now that some of our core API endpoints require multiple ordered requests to the graph DB to succeed. For example, searching for a study first retrieves that study and its relationships, but can then spur further calls for certain kinds of variations and therapies. This is totally reasonable for now but as requirements and the data models begin to settle, we should be aiming to condense those protocols into a single DB call eventually.
The text was updated successfully, but these errors were encountered: