-
Notifications
You must be signed in to change notification settings - Fork 810
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
Add remote write v2 proto #6320
base: master
Are you sure you want to change the base?
Add remote write v2 proto #6320
Conversation
// | ||
// For Go, see github.com/prometheus/prometheus/model/timestamp/timestamp.go | ||
// for conversion from/to time.Time to Prometheus timestamp. | ||
int64 timestamp = 15; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
At v2, the naming of unix timestamp is converted from timestamp_ms
to timestamp
for all of (exemplar, sample, and histogram).
So, I re-defined Histogram
instead of using the existing one.
322b5c9
to
6204558
Compare
6204558
to
6e0054a
Compare
Signed-off-by: SungJin1212 <[email protected]>
Signed-off-by: SungJin1212 <[email protected]>
6e0054a
to
ab3963d
Compare
Hello @SungJin1212, thank you for opening this PR. There is a release in progress. As such, please rebase your CHANGELOG entry on top of the master branch and move the CHANGELOG entry to the top under Thanks, |
Part of #6292.
This PR adds remote write v2 proto and
PreallocTimeseries
like v1.Which issue(s) this PR fixes:
Fixes #
Checklist
CHANGELOG.md
updated - the order of entries should be[CHANGE]
,[FEATURE]
,[ENHANCEMENT]
,[BUGFIX]