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

Handle formatting when the time in nanoseconds requires more than a single int64 #619

Merged
merged 1 commit into from
Jul 9, 2019

Conversation

jacksontj
Copy link
Contributor

@jacksontj jacksontj commented Jul 9, 2019

Signed-off-by: Thomas Jackson jacksontj.89@gmail.com

Fixup for #617

Fixes #614

cc @beorn7

…ingle int64

Signed-off-by: Thomas Jackson <jacksontj.89@gmail.com>

Fixup for prometheus#617
@jacksontj
Copy link
Contributor Author

cc @krasi-georgiev

@krasi-georgiev
Copy link
Contributor

LGTM

@jacksontj
Copy link
Contributor Author

@krasi-georgiev I assume we need someone else to do the actual merge, if so -- who? :)

@krasi-georgiev
Copy link
Contributor

usually that is @beorn7

@beorn7
Copy link
Member

beorn7 commented Jul 9, 2019

@krasi-georgiev is totally able and entitled to merge, too. :o)

@beorn7 beorn7 merged commit ff1d4e2 into prometheus:master Jul 9, 2019
@jacksontj jacksontj deleted the time_formatting branch July 9, 2019 20: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.

API client Series() call can't handle minTime/maxTime set by prometheus
3 participants