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

Should we ensure timeseries are in time order? #323

Open
pete-rai opened this issue May 20, 2022 · 0 comments
Open

Should we ensure timeseries are in time order? #323

pete-rai opened this issue May 20, 2022 · 0 comments
Labels
question Further information is requested

Comments

@pete-rai
Copy link
Collaborator

Time series should be returned from webhooks with latest first. This is because we want this to always be the latest node:

/timeseries/foo?limit=1

Should we ensure this in timeseries validation? validate.timeseries_records

Maybe we should, but then only log a warning on the connector?

@pete-rai pete-rai added the question Further information is requested label May 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

1 participant