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

Fix two small issues #825

Merged
merged 2 commits into from
Nov 2, 2016
Merged

Fix two small issues #825

merged 2 commits into from
Nov 2, 2016

Conversation

rkuchan
Copy link
Contributor

@rkuchan rkuchan commented Oct 21, 2016

Adds a question to the FAQ about the min and max allowable timestamps in the database. Also adds this information to the line protocol pages and documents the error that occurs when those timestamps are exceeded. Fixes #467.

Adds the max series per database exceeded error to the errors page. Fixes #819.

@beckettsean, I didn't see the write failed for shard XX text in the max series per database exceeded error. I tested it out on OSS 1.0.2.

falls outside the valid time range for InfluxDB.

The minimum valid timestamp is `-9223372036854775806` or `1677-09-21T00:12:43.145224194Z`.
The maximum valid timestamp is `9223372036854775806` or `2262-04-11T23:47:16.854775806Z`.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

According to https://golang.org/pkg/math/, in Go MaxInt64 is 1<<63 - 1, which according to wolfram alpha is 9223372036854775807.

Min is correct, since it's the actual Golang MinInt64 + 2.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay, it does look like the max you have is correct.

@beckettsean beckettsean merged commit 8e181c9 into master Nov 2, 2016
rkuchan added a commit that referenced this pull request Nov 2, 2016
@beckettsean beckettsean deleted the rk-some-issues branch November 7, 2016 22:59
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.

2 participants