Fix intersection

[kolesnikovae]

Although the new implementation is pretty expensive, it should not be an issue given that it is still in order of microseconds (and will never be the main factor of querying performance). I also think we should migrate to bitmaps for better set operations.

[codecov]

Codecov Report

Merging #425 (5157bb2) into main (6eb4a6c) will decrease coverage by 0.23%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #425      +/-   ##
==========================================
- Coverage   51.62%   51.40%   -0.22%     
==========================================
  Files         125      125              
  Lines        5713     5695      -18     
==========================================
- Hits         2949     2927      -22     
- Misses       2478     2482       +4     
  Partials      286      286              

Impacted Files	Coverage Δ
pkg/storage/dimension/dimension.go	45.72% <100.00%> (-15.64%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6eb4a6c...5157bb2. Read the comment docs.

[pyroscopebot]

Parameters

Details

Name	Value
BENCH_RUN_FOR	10m
PYROBENCH_RAND_SEED	2306912
PYROBENCH_PROFILE_WIDTH	20
PYROBENCH_PROFILE_DEPTH	20
PYROBENCH_PROFILE_SYMBOL_LENGTH	30
PYROBENCH_APPS	20
PYROBENCH_CLIENTS	20
PYROBENCH_REQUESTS	10000

Result

	main	pr	diff	threshold
throughput	155.91	155.63	-0.28 (-0.18%)	5%
total items processed	130386.00	130807.00	421.00 (0.32%)	5%

Details

Name	Description	Query for main	Query for pr
throughput		rate(pyroscope_http_request_duration_seconds_count{handler="/ingest", instance="pyroscope:4040"}[5m])	rate(pyroscope_http_request_duration_seconds_count{handler="/ingest", instance="pyroscope_main:4040"}[5m])
total items processed		pyroscope_http_request_duration_seconds_count{handler="/ingest", instance="pyroscope:4040"}	pyroscope_http_request_duration_seconds_count{handler="/ingest", instance="pyroscope_main:4040"}

Screenshots

Throughput

Disk Usage

Memory

Upload Errors (Total)

Successful Uploads (Total)

CPU Utilization

Generated by 🚫 dangerJS against a9be078

[petethepig]

Thanks for fixing this Anton!