Skip to content
This repository has been archived by the owner on Nov 8, 2022. It is now read-only.

Better exception handling based on user experience on long write #47

Closed
mbbroberg opened this issue Mar 8, 2016 · 1 comment
Closed

Comments

@mbbroberg
Copy link
Contributor

Notes from intelsdi-x/snap#731 -

  • @thomastaylor312 found that err="NaN is an unsupported value for field value" plugin-name=influx plugin-type=publisher plugin-version=10 point=<nil> caused a nil pointer reference and caused the plugin to panic. So a mixture of bad data and a hard to read log (it was all in byte arrays)
  • @geauxvirtual notes that this brings up a valid point that unexpected EOF could be improved upon. While that may be an error message received, it just becomes noise and has little value in knowing exactly why the job failed.
@mbbroberg mbbroberg added the bug label Jul 29, 2016
Alrim42 added a commit to Alrim42/snap-plugin-publisher-influxdb that referenced this issue Jul 21, 2017
marcintao added a commit that referenced this issue Jul 21, 2017
Closes #47, updated medium test
@Alrim42
Copy link
Contributor

Alrim42 commented Jul 21, 2017

To sum up:

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

No branches or pull requests

3 participants