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.

Sign up for GitHub

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

Jump to bottom

WIP: Delete values #325

Closed

Floppy wants to merge 3 commits into master from delete-values

Contributor

Floppy commented Dec 20, 2017 •

edited

Resolves #290. Needs tests adding and a bit more tidyup, but it's basically working.

Floppy added 3 commits

December 20, 2017 12:10


          delete single value

e5b79d7


          we can’t search on time for some reason, so we have to filter after

9be67c5


          try deleting

5f35dda

Floppy deployed to apolitical-metrics

December 20, 2017 12:47

View deployment

PaddyAlton commented Dec 21, 2017

New Delete method is overzealous in matching metrics to a timestamp - rather than exact matching it deletes all metrics from that day

(at least with timestamp supplied in format "yyyy-mm-ddThh:00:00.000000000" - everything stored for yyyy-mm-dd is removed)

Floppy closed this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment