-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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 & apply what time range query without stores should announce #1299
Comments
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
This issue/PR has been automatically marked as stale because it has not had recent activity. Please comment on status otherwise the issue will be closed in a week. Thank you for your contributions. |
We have yet to decide on this. |
This issue/PR has been automatically marked as stale because it has not had recent activity. Please comment on status otherwise the issue will be closed in a week. Thank you for your contributions. |
I like |
Happy with 0, 0. Will do a PR. |
This is a follow up from #1284 (comment)
If a query component has no stores discovered, it currently announces that it has data from time range
0
tomath.MaxInt64
. @bwplotka mentioned that this should either be all of time, which would bemath.MinInt64
tomath.MaxInt64
or nothing, which could be represented asmath.MaxInt64
tomath.MaxInt64
.I think either of those two make more sense than what we currently have, but I'm tending towards the latter, to not do unneeded requests to a store.
The text was updated successfully, but these errors were encountered: