[Hawkular-commits] [hawkular/hawkular-metrics] 476254: HWKMETRICS-322 InfluxDB API: groupby should accept...

[Stefan Negrea]

  Branch: refs/heads/master
  Home:   https://github.com/hawkular/hawkular-metrics
  Commit: 4762544f972f6b39229fe81d117f0f7d86b34ea4
      https://github.com/hawkular/hawkular-metrics/commit/4762544f972f6b39229fe81d117f0f7d86b34ea4
  Author: Thomas Segismont <tsegismo at redhat.com>
  Date:   2015-10-27 (Tue, 27 Oct 2015)

  Changed paths:
    M api/diff.txt
    M api/metrics-api-jaxrs/src/main/antlr4/org/hawkular/metrics/api/jaxrs/influx/query/parse/InfluxQuery.g4
    M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/influx/query/parse/definition/SelectQueryDefinitionsParser.java
    M api/metrics-api-jaxrs/src/test/java/org/hawkular/metrics/api/jaxrs/influx/query/parse/definition/SelectQueryGroupByClauseTest.java
    M api/metrics-api-jaxrs/src/test/resources/influx/query/syntactically-correct-queries.iql
    M api/metrics-api-jaxrs/src/test/resources/influx/query/syntactically-incorrect-queries.iql

  Log Message:
  -----------
  HWKMETRICS-322 InfluxDB API: groupby should accept milliseconds unit

Updated the grammar and tests. I thought ms were not accepted in groupby clause, as indicated in the documentation:
https://influxdb.com/docs/v0.8/api/query_language.html#group-by

But running the query on a real Influx server shows that it is supported.


  Commit: 5bdfd8c10377dd669bdbbfbead68461bb3e4c3f7
      https://github.com/hawkular/hawkular-metrics/commit/5bdfd8c10377dd669bdbbfbead68461bb3e4c3f7
  Author: Stefan Negrea <snegrea at redhat.com>
  Date:   2015-10-27 (Tue, 27 Oct 2015)

  Changed paths:
    M api/diff.txt
    M api/metrics-api-jaxrs/src/main/antlr4/org/hawkular/metrics/api/jaxrs/influx/query/parse/InfluxQuery.g4
    M api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/influx/query/parse/definition/SelectQueryDefinitionsParser.java
    M api/metrics-api-jaxrs/src/test/java/org/hawkular/metrics/api/jaxrs/influx/query/parse/definition/SelectQueryGroupByClauseTest.java
    M api/metrics-api-jaxrs/src/test/resources/influx/query/syntactically-correct-queries.iql
    M api/metrics-api-jaxrs/src/test/resources/influx/query/syntactically-incorrect-queries.iql

  Log Message:
  -----------
  Merge pull request #405 from tsegismont/jira/HWKMETRICS-322

HWKMETRICS-322 InfluxDB API: groupby should accept milliseconds unit


Compare: https://github.com/hawkular/hawkular-metrics/compare/9024aa2a517e...5bdfd8c10377