Skip to content

Commit

Permalink
Update the instrument agg comp table (#4330)
Browse files Browse the repository at this point in the history
Use a check mark to indicate compatible instead of an "X" which can be
misinterpreted as incompatible.
  • Loading branch information
MrAlias authored Jul 17, 2023
1 parent 7467923 commit 63dfc4c
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions sdk/metric/pipeline.go
Original file line number Diff line number Diff line change
Expand Up @@ -425,12 +425,12 @@ func (i *inserter[N]) aggregateFunc(b aggregate.Builder[N], agg aggregation.Aggr
//
// | Instrument Kind | Drop | LastValue | Sum | Histogram | Exponential Histogram |
// |--------------------------|------|-----------|-----|-----------|-----------------------|
// | Counter | X | | X | X | X |
// | UpDownCounter | X | | X | | |
// | Histogram | X | | X | X | X |
// | Observable Counter | X | | X | | |
// | Observable UpDownCounter | X | | X | | |
// | Observable Gauge | X | X | | | |.
// | Counter | | | | | |
// | UpDownCounter | | | | | |
// | Histogram | | | | | |
// | Observable Counter | | | | | |
// | Observable UpDownCounter | | | | | |
// | Observable Gauge | | | | | |.
func isAggregatorCompatible(kind InstrumentKind, agg aggregation.Aggregation) error {
switch agg.(type) {
case aggregation.Default:
Expand Down

0 comments on commit 63dfc4c

Please sign in to comment.