[pull] master from statsd:master #1
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
![pull[bot]](https://avatars.githubusercontent.com/in/12910?s=60&v=4){kind=small}
Setting keyNameSanitize to false pushes the requirement of sanitizing key names to the backends. This permits backends that have less strict character set requirements to take advantage of an expanded stat name character set. The default behavior remains the same as collisions in key name space are not handled if two different stat names map to the same sanitized key name.
…o pr_155_take2 Conflicts: stats.js
…ponsibility of each backend developer to take care for sanitization
Pretty was using an invalid invocation of util.inspect(). Perhaps it was valid pre-v0.10, but inspect now takes an options object.
Added InfluxDB (assuming alphabetical order)
add netuitive backend to backends.md
Add node-bell to statsd backend list
Enable packaging at https://packager.io/gh/pkgr/statsd
Fix prettyprint option for console backend
Validate negative sampling
Describe timer sample rate in metric_types docs. Fixes #435
This reverts commit 0eb2517.
Correct spelling mistake in backend.md
* support for tagged metrics * decode datadog tag format * don't replace tagging characters when sanitizing graphite metric names
When this was being set to the startup_time it was causing confusion with users of the admin panel, leading people to believe there had been an exception which wasn't being logged. Since no exceptions are occuring, a 0 initialisation seems reasonable here.
There are small typos in: - docs/graphite.md - docs/metric_types.md - exampleConfig.js - examples/python_example.py - proxy.js Fixes: - Should read `occurrences` rather than `occurences`. - Should read `undefined` rather than `undefinded`. - Should read `signifying` rather than `signfying`. - Should read `retrieves` rather than `retreives`. - Should read `retrieved` rather than `retreived`. - Should read `resources` rather than `recources`. - Should read `metric` rather than `mertric`.
The Docker Compose syntax has changed since this file was added. This makes the configuration actually work with modern docker-compose.
Fixed docker-compose configuration for 2
Update Node base for the Docker image
Images are published to dockerhub and ghcr as documented in the action file.
This reverts commit bdca799.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
See Commits and Changes for more details.
Created by
pull[bot]
Can you help keep this open source service alive? 💖 Please sponsor : )