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

Jump to bottom

Limit max execution time of transitions feature #15609

Closed
tsteur opened this issue Feb 21, 2020 · 0 comments · Fixed by #15652
Closed

Limit max execution time of transitions feature #15609

tsteur opened this issue Feb 21, 2020 · 0 comments · Fixed by #15652
Assignees
@sgiehl
Labels
c: Performance For when we could improve the performance / speed of Matomo.
Milestone
3.13.4

Comments

@tsteur
Copy link
Member

tsteur commented Feb 21, 2020 •
edited

We should apply live_query_max_execution_time setting also for transition queries since it is basically also a live query. Ideally, if the limit is reached, similar to visitor log we would then show a message if the limit is reached and suggest to shorten the date range for example, or if a segment is selected mention could be caused by this.

refs #14762
@tsteur tsteur added the c: Performance For when we could improve the performance / speed of Matomo. label Feb 21, 2020
@tsteur tsteur added this to the 3.13.4 milestone Feb 21, 2020
@sgiehl sgiehl self-assigned this Mar 3, 2020
@sgiehl sgiehl mentioned this issue Mar 3, 2020
Merged
@sgiehl sgiehl linked a pull request Mar 3, 2020 that will close this issue
respect max execution time limit for transitions plugin #15652
Merged
@tsteur tsteur closed this as completed Mar 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sgiehl sgiehl

Labels
c: Performance For when we could improve the performance / speed of Matomo.
Projects
None yet
Milestone
3.13.4
Development

Successfully merging a pull request may close this issue.

respect max execution time limit for transitions plugin matomo-org/matomo
2 participants
@tsteur @sgiehl