[Hawkular-commits] [hawkular/hawkular-metrics] 307975: some minor updates to run/test against C* 2.2.0

[Thomas Segismont]

  Branch: refs/heads/master
  Home:   https://github.com/hawkular/hawkular-metrics
  Commit: 307975d17dc1e33bfcdf1ee89fa8c8de37369a97
      https://github.com/hawkular/hawkular-metrics/commit/307975d17dc1e33bfcdf1ee89fa8c8de37369a97
  Author: John Sanda <jsanda at redhat.com>
  Date:   2015-08-27 (Thu, 27 Aug 2015)

  Changed paths:
    M .travis.install.cassandra.sh
    M core/metrics-core-impl/src/main/java/org/hawkular/metrics/core/impl/transformers/BatchStatementTransformer.java
    M core/metrics-core-impl/src/test/java/org/hawkular/metrics/core/impl/transformers/BatchStatementTransformerTest.java
    M rest-tests/src/test/groovy/org/hawkular/metrics/rest/CountersITest.groovy
    M rest-tests/src/test/groovy/org/hawkular/metrics/rest/GaugeMetricStatisticsITest.groovy
    M rest-tests/src/test/groovy/org/hawkular/metrics/rest/RESTTest.groovy

  Log Message:
  -----------
  some minor updates to run/test against C* 2.2.0

With this commit, everyone should be using Cassandra 2.2.0 in their local dev
environments.

In additon to having Travis use C* 2.2.0, I had to make some changes around
batch statements. When testing locally with a 2.2.0 cluster, I was getting some
failures about exceeding the batch_size_fail_threshold_in_kb setting defined
in cassandra.yaml. It defaults to 50 KB. BatchStatementTransformer had a check
on the number of statements in a batch. This is not really useful though
because the check needs to be on the overall size, not the number of statements
in the batch. I am going to create a ticket to address our usage of batch
statements. The conventional wisdom in general is to avoid them, and it makes
sense for adhere to that.


  Commit: 729fde55e890e25c62f09c049a7dabded5520ee0
      https://github.com/hawkular/hawkular-metrics/commit/729fde55e890e25c62f09c049a7dabded5520ee0
  Author: John Sanda <jsanda at redhat.com>
  Date:   2015-08-27 (Thu, 27 Aug 2015)

  Changed paths:
    M .travis.install.cassandra.sh

  Log Message:
  -----------
  fix travis script to download 2.2.x release


  Commit: e123390d60e4fa9be61fb419ec3d89068184b5fa
      https://github.com/hawkular/hawkular-metrics/commit/e123390d60e4fa9be61fb419ec3d89068184b5fa
  Author: John Sanda <jsanda at redhat.com>
  Date:   2015-08-27 (Thu, 27 Aug 2015)

  Changed paths:
    M .travis.install.cassandra.sh

  Log Message:
  -----------
  remove line break added by intellij


  Commit: 51d8cc3148e5412107dbb6c7cbe3b5fb0146489c
      https://github.com/hawkular/hawkular-metrics/commit/51d8cc3148e5412107dbb6c7cbe3b5fb0146489c
  Author: John Sanda <jsanda at redhat.com>
  Date:   2015-08-31 (Mon, 31 Aug 2015)

  Changed paths:
    M core/metrics-core-impl/src/main/java/org/hawkular/metrics/core/impl/transformers/BatchStatementTransformer.java
    M core/metrics-core-impl/src/test/java/org/hawkular/metrics/core/impl/transformers/BatchStatementTransformerTest.java
    M rest-tests/src/test/groovy/org/hawkular/metrics/rest/GaugeMetricStatisticsITest.groovy
    M rest-tests/src/test/groovy/org/hawkular/metrics/rest/RESTTest.groovy

  Log Message:
  -----------
  reduce max batch size and revert other changes based on PR feedback


  Commit: 59b8ccdf23a3353373b546d08ae551a6000e8d8c
      https://github.com/hawkular/hawkular-metrics/commit/59b8ccdf23a3353373b546d08ae551a6000e8d8c
  Author: Thomas Segismont <tsegismont at gmail.com>
  Date:   2015-09-03 (Thu, 03 Sep 2015)

  Changed paths:
    M .travis.install.cassandra.sh
    M core/metrics-core-impl/src/main/java/org/hawkular/metrics/core/impl/transformers/BatchStatementTransformer.java
    M rest-tests/src/test/groovy/org/hawkular/metrics/rest/CountersITest.groovy
    M rest-tests/src/test/groovy/org/hawkular/metrics/rest/RESTTest.groovy

  Log Message:
  -----------
  Merge pull request #329 from jsanda/cassandra-2.2

some minor updates to run/test against C* 2.2.0


Compare: https://github.com/hawkular/hawkular-metrics/compare/f17609a6619e...59b8ccdf23a3