[Hawkular-commits] [hawkular/hawkular-metrics] f131ad: [HWKMETRICS-204] fix schema and implicit tenant cr...

[John Sanda]

  Branch: refs/heads/implicit-tenants
  Home:   https://github.com/hawkular/hawkular-metrics
  Commit: f131ad31124310fabfde3b75d954143902e6bf05
      https://github.com/hawkular/hawkular-metrics/commit/f131ad31124310fabfde3b75d954143902e6bf05
  Author: John Sanda <jsanda at redhat.com>
  Date:   2015-08-13 (Thu, 13 Aug 2015)

  Changed paths:
    M core/metrics-core-api/src/main/java/org/hawkular/metrics/core/api/MetricsService.java
    M core/metrics-core-impl/src/main/java/org/hawkular/metrics/core/impl/CreateTenants.java
    M core/metrics-core-impl/src/main/java/org/hawkular/metrics/core/impl/DataAccess.java
    M core/metrics-core-impl/src/main/java/org/hawkular/metrics/core/impl/DataAccessImpl.java
    M core/metrics-core-impl/src/main/java/org/hawkular/metrics/core/impl/MetricsServiceImpl.java
    A core/metrics-core-impl/src/main/java/org/hawkular/metrics/core/impl/TenantsService.java
    M core/metrics-core-impl/src/test/java/org/hawkular/metrics/core/impl/CreateTenantsITest.java
    M core/metrics-core-impl/src/test/java/org/hawkular/metrics/core/impl/DelegatingDataAccess.java
    M schema-manager/src/main/resources/schema.cql

  Log Message:
  -----------
  [HWKMETRICS-204] fix schema and implicit tenant creation logic

The tenant column in tenants_by_time needs to be a clustering column.
CreateTenants has been updated to properly filter out id of tenants that
already exist. A new tenant creation method in MetricsServiceImpl is used. It
does not use a light weight transaction, nor does it update the retentions
index. There is no reason to incur the overhead of a LWT because its use is
intended for explicit tenant creation to ensure there is no naming conflict.