[GitHub] cloudstack pull request: CLOUDSTACK-8300: Set indexes on event tab...

2016-01-31 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/cloudstack/pull/1387


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-8300: Set indexes on event tab...

2016-01-31 Thread wido
Github user wido commented on the pull request:

https://github.com/apache/cloudstack/pull/1387#issuecomment-177549166
  
Yes, simple but sane fix indeed. LGTM


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-8300: Set indexes on event tab...

2016-01-30 Thread bhaisaab
Github user bhaisaab commented on the pull request:

https://github.com/apache/cloudstack/pull/1387#issuecomment-177242702
  
LGTM


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] cloudstack pull request: CLOUDSTACK-8300: Set indexes on event tab...

2016-01-30 Thread remibergsma
GitHub user remibergsma opened a pull request:

https://github.com/apache/cloudstack/pull/1387

CLOUDSTACK-8300: Set indexes on event table

When there are many events in the cloud.event table, the UI throws an SQL 
exception and the management server spikes at 100% CPU for minutes. That causes 
other API calls to fail with 530 errors.

It seems an index on the 'archived' field is missing (since it is used in 
the WHERE clause).

We have 1.4M events in the table:
`select count(*) from event;`
1497838

This PR adds the index to the `archived` field and the `state` field.


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/remibergsma/cloudstack CLOUDSTACK-8300

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/1387.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1387


commit 14c837ab8329f98b3fe071dd69807dcae77f4f7c
Author: Remi Bergsma 
Date:   2016-01-30T15:50:46Z

CLOUDSTACK-8300: Set indexes on event table




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---