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

resty tracking issue #26

Open
6 of 7 tasks
SamStudio8 opened this issue Jul 1, 2020 · 2 comments
Open
6 of 7 tasks

resty tracking issue #26

SamStudio8 opened this issue Jul 1, 2020 · 2 comments
Assignees

Comments

@SamStudio8
Copy link
Owner

SamStudio8 commented Jul 1, 2020

  • update v3 to use new institute credits - eventually migrate default credit to same system

  • check the actual mdv permission as well as the permission key

  • check the new permissions model works on list as well as retrieve (cant we just override get?)

  • add permissions to tatlauthenticator

  • add accesses and time taken metrics to dataview, return average wait time to client

  • basic example

  • intergrate with celery

@SamStudio8 SamStudio8 self-assigned this Jul 1, 2020
@SamStudio8
Copy link
Owner Author

add accesses and time taken metrics to dataview, return average wait time to client

this is done by the tatltask now

@SamStudio8
Copy link
Owner Author

The v3 API is a great success in that its quite a technical achievement. Sadly the performance is garbage, as DRF exacerbates the n+1 query problem. This isn't a fault of DRF, just the implementation is in contradiction to the highly interconnected artifacts and processes in Majora, which adds a significant query overhead. No doubt this could be improved somewhat with careful use of prefetching in the right places but I think we won't be able to move forward with v3 without a sensible cache in front of the groups (if not the artifacts too).

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

No branches or pull requests

1 participant