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

schema: Add indices needed for detail histories in Web #367

Closed
wants to merge 1 commit into from
Closed

schema: Add indices needed for detail histories in Web #367

wants to merge 1 commit into from

Conversation

nilmerg
Copy link
Member

@nilmerg nilmerg commented Sep 21, 2021

@cla-bot cla-bot bot added the cla/signed label Sep 21, 2021
@nilmerg nilmerg marked this pull request as draft September 21, 2021 15:12
@nilmerg
Copy link
Member Author

nilmerg commented Sep 21, 2021

@lippserd This speeds up the detail histories significantly, but will mess up the main history. It ignores idx_history_event_time and uses now idx_history_host_service_id_object_type_event_time instead. If I add an index hint (FROM history USE INDEX (idx_history_event_time)), the main history is back to normal speeds but I don't know if that's the only way to solve this. Please take a look here. You'll need Icinga/icingadb-web#364 for this.

@julianbrost julianbrost added the needs-feedback We'll only proceed once we hear from you again label Sep 27, 2021
@nilmerg nilmerg removed the needs-feedback We'll only proceed once we hear from you again label Nov 9, 2021
@nilmerg nilmerg closed this Nov 9, 2021
@nilmerg nilmerg deleted the add-history-indices-needed-for-web branch November 9, 2021 12:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
cla/signed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@nilmerg @julianbrost @lippserd