Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for filtering attribute keys for streams via an exclude list #4188

Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 6 additions & 4 deletions specification/metrics/sdk.md
Original file line number Diff line number Diff line change
Expand Up @@ -364,10 +364,6 @@ The SDK MUST accept the following stream configuration parameters:
keys that identify the attributes that MUST be kept, and all other attributes
MUST be ignored.

Additionally, implementations SHOULD support configuring an exclude-list of
attribute keys. The exclude-list contains attribute keys that identify the
attributes that MUST be excluded, all other attributes MUST be kept.

Implementations MAY accept additional attribute filtering functionality for
this parameter.

Expand All @@ -379,6 +375,12 @@ The SDK MUST accept the following stream configuration parameters:
parameter configured on the instrument instead. If the `Attributes`
advisory parameter is absent, all attributes MUST be kept.

Additionally, implementations SHOULD support configuring an exclude-list of
attribute keys. The exclude-list contains attribute keys that identify the
attributes that MUST be excluded, all other attributes MUST be kept. If an
attribute key is both included and excluded, implementations MAY cause the
cijothomas marked this conversation as resolved.
Show resolved Hide resolved
application to fail on initialization.

* `aggregation`: The name of an [aggregation](#aggregation) function to use in
aggregating the metric stream data.

Expand Down
Loading