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

Decide how to handle different data types #5

Open
margau opened this issue Jun 4, 2018 · 3 comments
Open

Decide how to handle different data types #5

margau opened this issue Jun 4, 2018 · 3 comments

Comments

@margau
Copy link
Member

margau commented Jun 4, 2018

If we not have identical types in the same Channel, Grafana will burn down

@adnidor
Copy link

adnidor commented Jun 4, 2018

InfluxDB will burn down before Grafana.

Trying to write a value of a type not matching the previous types will result in an error

@margau
Copy link
Member Author

margau commented Jun 4, 2018

Maybe we should give the option to define the Channel-Types per Sensor in the control panel? If the first value for a new Sensor is in the wrong type (which is possible trough debugging a sensor), the sensor-id is lost.

@hibas123
Copy link
Member

Maybe we should give the option to define the Channel-Types per Sensor in the control panel? If the first value for a new Sensor is in the wrong type (which is possible trough debugging a sensor), the sensor-id is lost.

The data input should work as fast as possible. Any additional database access would create a large speed impact. I think the easiest way would be just to let the database create errors, what wouldn't impact the performance significantly. In case of an error, we could try an automatic type adjustment, and in case this is not working, the user should be informed. Although listing the type of the channel on the control panel would be a good idea.

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

No branches or pull requests

3 participants