Skip to content

allow changing jolokia delimiter#2255

Merged
sparrc merged 1 commit intoinfluxdata:masterfrom
phemmer:jolokia-delimiter
Jan 12, 2017
Merged

allow changing jolokia delimiter#2255
sparrc merged 1 commit intoinfluxdata:masterfrom
phemmer:jolokia-delimiter

Conversation

@phemmer
Copy link
Contributor

@phemmer phemmer commented Jan 12, 2017

Required for all PRs:

  • CHANGELOG.md updated (we recommend not updating this until the PR has been approved by a maintainer)
  • Sign CLA (if not already signed)

This allows changing the delimiter jolokia uses for concatenating metric names & attribute keys. It is very common for an attribute key to contain an underscore, thus using an underscore as the delimiter makes the field names rather ambiguous.

Personally I use a dot (.) as my delimiter, as it is very rare to see a dot in an attribute key, but I've been doing so by explicitly specifying each attribute as a separate [inputs.jolokia.metric].

@sparrc sparrc added this to the 1.2.0 milestone Jan 12, 2017
@sparrc sparrc merged commit b7d29ca into influxdata:master Jan 12, 2017
@phemmer phemmer deleted the jolokia-delimiter branch January 12, 2017 13:13
njwhite pushed a commit to njwhite/telegraf that referenced this pull request Jan 31, 2017
MrMaxBuilds pushed a commit that referenced this pull request Jun 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants