Skip to content

Conversation

mahendrapaipuri
Copy link
Contributor

@mahendrapaipuri mahendrapaipuri commented Feb 17, 2025

  • WithLookbackDelta and WithStats options have been added to API client.
  • lookback_delta can be especially handy when dealing with mix of continuous and intermittent time series.

* `WithLookbackDelta` and `WithPerStepStats` options have been added to API client.

Signed-off-by: Mahendra Paipuri <[email protected]>
Copy link
Member

@bwplotka bwplotka left a comment

Choose a reason for hiding this comment

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

Thanks, just one suggestion around stats.

mahendrapaipuri and others added 2 commits February 20, 2025 14:07
Co-authored-by: Bartlomiej Plotka <[email protected]>
Signed-off-by: Mahendra Paipuri <[email protected]>
* Use a custom `StatsValue` type to provide as query parameter for `stats` option.

Signed-off-by: Mahendra Paipuri <[email protected]>
Copy link
Member

@bwplotka bwplotka left a comment

Choose a reason for hiding this comment

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

Thanks!

@bwplotka bwplotka merged commit 6b820eb into prometheus:main Feb 21, 2025
9 checks passed
ying-jeanne pushed a commit to ying-jeanne/client_golang that referenced this pull request Mar 28, 2025
* refactor: Add support for undocumented query options for API

* `WithLookbackDelta` and `WithPerStepStats` options have been added to API client.

Signed-off-by: Mahendra Paipuri <[email protected]>

* Update api/prometheus/v1/api.go

Co-authored-by: Bartlomiej Plotka <[email protected]>
Signed-off-by: Mahendra Paipuri <[email protected]>

* refactor: Address PR comments and feedback

* Use a custom `StatsValue` type to provide as query parameter for `stats` option.

Signed-off-by: Mahendra Paipuri <[email protected]>

---------

Signed-off-by: Mahendra Paipuri <[email protected]>
Signed-off-by: Mahendra Paipuri <[email protected]>
Co-authored-by: Bartlomiej Plotka <[email protected]>
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.

3 participants