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

Metrics picker in Graphs which uses API.get should not trigger archiving #4755

Closed
mattab opened this issue Feb 25, 2014 · 1 comment
Closed
Labels
Bug For errors / faults / flaws / inconsistencies etc. c: Performance For when we could improve the performance / speed of Matomo. wontfix If you can reproduce this issue, please reopen the issue or create a new one describing it.
Milestone

Comments

@mattab
Copy link
Member

mattab commented Feb 25, 2014

when loading Visitors > Overview, the graph at the top of the page has a Metrics picker. This metrics picker uses API.get to get the list of metrics available to display. When the picker calls API.get, if we are currently looking at a custom date range (period=range) then the API.get will be triggered on all plugins APIs loaded by this magic API.

So in turn the VisitFrequency.get and Goals.get and Actions.get and VisitsSummary.get are triggered. This is slow when not pre-archived.

So we should fix so that the picker itself only calls the report Metadata api and not the ask for report data when we only need metrics names.

@mattab
Copy link
Member Author

mattab commented Mar 16, 2014

not needed because this was done: #4768

@mattab mattab added this to the 2.2.1 - Piwik 2.2.1 milestone Jul 8, 2014
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug For errors / faults / flaws / inconsistencies etc. c: Performance For when we could improve the performance / speed of Matomo. wontfix If you can reproduce this issue, please reopen the issue or create a new one describing it.
Projects
None yet
Development

No branches or pull requests

1 participant