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

Limit the results of the carbon query to only the current range #130

Merged
merged 1 commit into from
Oct 5, 2015

Conversation

jsternberg
Copy link
Contributor

The carbon query will return all cached datapoints. This includes values
that are outside of your current query. When the carbonlink attempts to
put these values in the returned array, an index error gets thrown. This
causes a lot of errors that get thrown when datapoints are cached and
the requested range does not match the timestamps of the cached
datapoints.

Teach the carbon query how to limit which metrics get inserted into the
cache based on their timestamps.

The carbon query will return all cached datapoints. This includes values
that are outside of your current query. When the carbonlink attempts to
put these values in the returned array, an index error gets thrown. This
causes a lot of errors that get thrown when datapoints are cached and
the requested range does not match the timestamps of the cached
datapoints.

Teach the carbon query how to limit which metrics get inserted into the
cache based on their timestamps.
@hamelg hamelg mentioned this pull request Sep 30, 2015
brutasse added a commit that referenced this pull request Oct 5, 2015
Limit the results of the carbon query to only the current range
@brutasse brutasse merged commit ba8a301 into brutasse:master Oct 5, 2015
@brutasse
Copy link
Owner

brutasse commented Oct 5, 2015

Thanks!

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 this pull request may close these issues.

None yet

2 participants