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

Omission of a required Field should trigger an exception rather than failing silently #149

Open
baptgb opened this issue Sep 2, 2023 · 0 comments · May be fixed by #150
Open

Omission of a required Field should trigger an exception rather than failing silently #149

baptgb opened this issue Sep 2, 2023 · 0 comments · May be fixed by #150

Comments

@baptgb
Copy link

baptgb commented Sep 2, 2023

Proposal:
Introduce a distinct exception for attempts to write a Point without a field. This would facilitate debugging, accelerate the learning curve for InfluxDB's key concepts, and provide a transparent error message.

Current behavior:
Writing a Point without a field currently results in silent failure, with the Point being transformed to null and no indication that the Point has been disregarded.

Desired behavior:
Trigger an exception when a developer attempts to write a Point with no field.

Alternatives considered:
Display a clear warning to indicate that the Point is being ignored due to the absence of a field.

Use case:
InfluxDB mandates that each Point must contain at least one field. This requirement might not be intuitive for newcomers. Thus, generating an exception for such cases would significantly reduce time and frustration when initially working with the database.
In my experience, I had to debug the client's code to understand why my Point wasn't being written. Although it's essential to have at least one field, this wasn't immediately clear to me, leading to unnecessary confusion.

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

Successfully merging a pull request may close this issue.

1 participant