Skip to content
This repository has been archived by the owner on Jul 19, 2023. It is now read-only.

Validates that push request contains at least one sample #550

Merged
merged 1 commit into from
Feb 27, 2023
Merged

Validates that push request contains at least one sample #550

merged 1 commit into from
Feb 27, 2023

Conversation

cyriltovena
Copy link
Collaborator

Fixes #543

simonswine
simonswine approved these changes Feb 27, 2023
View reviewed changes
Copy link
Collaborator

@simonswine simonswine left a comment

Choose a reason for hiding this comment

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

LGTM

@cyriltovena cyriltovena merged commit 46b7b5f into main Feb 27, 2023
@cyriltovena cyriltovena deleted the validate-push-sample branch February 27, 2023 14:01
simonswine pushed a commit to simonswine/pyroscope that referenced this pull request Jun 30, 2023
@cyriltovena
Validates that push request contains at least one sample (grafana/phl…
ca5bb7f 
…are#550)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@simonswine simonswine simonswine approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Distributor hangs with empty push requests
2 participants
@cyriltovena @simonswine