-
Notifications
You must be signed in to change notification settings - Fork 244
Open
Labels
area/storageSomething about the storageSomething about the storageenhancementNew feature or requestNew feature or request
Description
Most of the work needs to be done in FrostDB: polarsignals/frostdb#347
Lines 571 to 579 in ed90dbe
| // TODO: This still queries way too much data from the underlying database. | |
| // This needs to be moved to FrostDB to not even query all of this data in the first place. | |
| // With a scrape interval of 10s and a query range of 1d we'd query 8640 samples and at most return 960. | |
| // Even worse for a week, we'd query 60480 samples and only return 1000. | |
| tsBucket := ts / 1000 / int64(step.Seconds()) | |
| if _, found := resSeriesBuckets[index][tsBucket]; found { | |
| // We already have a MetricsSample for this timestamp bucket, ignore it. | |
| continue | |
| } |
This issue is mostly to keep track of updating Parca once the necessary parts have been implemented in FrostDB.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
area/storageSomething about the storageSomething about the storageenhancementNew feature or requestNew feature or request