Skip to content

Commit

Permalink
Merge pull request #23 from statful/TEL-2043-add-p99-aggregation
Browse files Browse the repository at this point in the history
chore(TEL-2043): Add p99 aggregation
  • Loading branch information
FranciscoMota authored Nov 19, 2018
2 parents afad51f + c035259 commit b8c50d2
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -237,7 +237,7 @@ Read the methods options reference bellow to get more information about the defa
| Description | Default for Counter | Default for Gauge | Default for Timer | Default for Put | Available for Aggregated Methods |
|:---|:---|:---|:---|:---|:---|
| **_agg_** (`array`) - Defines the aggregations to be executed. These aggregations are merged with the ones configured globally, including method defaults.<br><br> **Valid Aggregations:** `avg, count, sum, first, last, p90, p95, min, max` | `['sum', 'count']` | `[last]` | `['avg', 'p90', 'count']` | `[]` | **NO** |
| **_agg_** (`array`) - Defines the aggregations to be executed. These aggregations are merged with the ones configured globally, including method defaults.<br><br> **Valid Aggregations:** `avg, count, sum, first, last, p90, p95, p99, min, max` | `['sum', 'count']` | `[last]` | `['avg', 'p90', 'count']` | `[]` | **NO** |
| **_aggFreq_** (`number`) - Defines the aggregation frequency in **seconds**. It overrides the global aggregation frequency configuration.<br><br> **Valid Aggregation Frequencies:** `10, 30, 60, 120, 180, 300` | `10` | `10` | `10` | `10`' | **NO** |
| **_namespace_** (`string`) - Defines the namespace of the metric. It overrides the global namespace configuration. | `application` | `application` | `application` | `application` | **YES** |
| **_tags_** (`object`) - Defines the tags of the metric. These tags are merged with the ones configured globally, including method defaults. | `{}` | `{}` | `{ unit: 'ms' }` | `{}` | **YES** |
Expand Down
2 changes: 1 addition & 1 deletion lib/config-helper.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

var concat = require('unique-concat');

var validAggregations = ['avg', 'sum', 'count', 'first', 'last', 'p90', 'p95', 'min', 'max'];
var validAggregations = ['avg', 'sum', 'count', 'first', 'last', 'p90', 'p95', 'p99', 'min', 'max'];
var validAggregationFrequencies = [10, 30, 60, 120, 180, 300];

function isInteger (number) {
Expand Down

0 comments on commit b8c50d2

Please sign in to comment.