Not allowing invalid datapoints to ruin entire batches #1779

Merged
merged 2 commits into from Jun 16, 2016

Projects

None yet

1 participant

@captncraig
Contributor
  1. opentsdb.DataPoint.IsValid is largely unchanged. Determines if datapoint is valid to submit as is.
  2. If collect gets a non-cleanable datapoint it silently drops it.
  3. When you Add a datapoint, if the tags cannot be cleaned, it will log a message with (hopefully) a helpful line number, and drop it.
captncraig added some commits Jun 13, 2016
@captncraig captncraig Not allowing invalid datapoints to ruin entire batches
1. opentsdb.DataPoint.IsValid is largely unchanged. Determines if datapoint is valid to submit as is.
2. If collect gets a non-cleanable datapoint it silently drops it.
3. When you `Add` a datapoint, if the tags cannot be cleaned, it will log a message with (hopefully) a helpful line number, and drop it.
bf423c0
@captncraig captncraig Fix linux
fc7a532
@captncraig captncraig merged commit 7396a3a into master Jun 16, 2016

2 checks passed

bosun All checks Passed!
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@kylebrandt kylebrandt deleted the cleanOnAdd branch Jul 26, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment