[Hawkular-commits] [hawkular/hawkular-alerts] 4da35f: HWKALERTS-201 Improve scale for getAlerts/getEvent...

[Jay Shaughnessy]

  Branch: refs/heads/master
  Home:   https://github.com/hawkular/hawkular-alerts
  Commit: 4da35fab98941f1ca9f2ac7efd38a18d3bc1f47e
      https://github.com/hawkular/hawkular-alerts/commit/4da35fab98941f1ca9f2ac7efd38a18d3bc1f47e
  Author: Jay Shaughnessy <jshaughn at users.noreply.github.com>
  Date:   2016-11-16 (Wed, 16 Nov 2016)

  Changed paths:
    M hawkular-alerts-api/src/main/java/org/hawkular/alerts/api/services/EventsCriteria.java
    M hawkular-alerts-engine/src/main/java/org/hawkular/alerts/engine/impl/CassAlertsServiceImpl.java
    M hawkular-alerts-engine/src/main/java/org/hawkular/alerts/engine/impl/CassStatement.java
    M hawkular-alerts-engine/src/main/resources/org/hawkular/alerts/schema/cassalog.groovy
    M hawkular-alerts-engine/src/main/resources/org/hawkular/alerts/schema/checker.cql
    A hawkular-alerts-engine/src/main/resources/org/hawkular/alerts/schema/updates/schema-1.4.0.groovy
    M hawkular-alerts-engine/src/test/java/org/hawkular/alerts/engine/CassPersistenceTest.java
    M hawkular-alerts-engine/src/test/java/org/hawkular/alerts/engine/PersistenceTest.java

  Log Message:
  -----------
  HWKALERTS-201 Improve scale for getAlerts/getEvents (#261)

* HWKALERTS-201 Improve scale for getAlerts/getEvents
For getAlerts:
- remove indexing on status and severity
  - the low cardinality for these fields make them bad for
    indexing as the number alerts grows.  And for filtering the
    number of returned alertIds will be very large.
  - prior to filtering on these fields, fetch the alerts (typically
    already filtered significantly by time) and then filter them
    directly.
For getEvents
  - although category may have low cardinality it is unknown because
    categories are set by the client. So, maintain the index.
  - filter on category either via query+intersection, or manually,
    based on the size of the event set already filtered.
  - make the CriteriaNoQuerySize configurable and also optiponally
    overridden in the criteria.
- Fix issue when initial filter returns 0 entities
- Add some tests

* PR Feedback