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

Exclude non parsed data points #26

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

brupm
Copy link

@brupm brupm commented May 30, 2015

No description provided.

@suyograo
Copy link
Contributor

suyograo commented Jun 5, 2015

@brupm is there any way to add tests, even using mocks?

Also this is an extension from this PR right? #11

@jordansissel
Copy link
Contributor

Code seems OK, but I have no experience with influxdb, so I'm not sure how to test. Also missing tests as @suyograo mentioned.

@brupm Can you work on this?

@brupm
Copy link
Author

brupm commented Sep 28, 2015

I really want to. I've been meaning to spend some time finishing up these PRs but work's been taking up all of my time. Give a couple weeks.

@elasticsearch-release
Copy link

Jenkins standing by to test this. If you aren't a maintainer, you can ignore this comment. Someone with commit access, please review this and clear it for Jenkins to run; then say 'jenkins, test it'.

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

Successfully merging this pull request may close these issues.

4 participants