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

Conversation

simonswine
Copy link
Collaborator

Allow reading of compressed and uncompressed data whenever we read a
profile from bytes through shared code. Also reader/writers and buffers
are pooled.

@simonswine simonswine added the kind/enhancement New feature or request label Oct 20, 2022
simonswine added a commit that referenced this pull request Oct 20, 2022
This is because the mapping ID is set to 0 on the locations.

It also adds an uncompressed profile that should be solved by #338.
Allow reading of compressed and uncompressed data whenever we read a
profile from bytes through shared code. Also reader/writers and buffers
are pooled.
@simonswine simonswine force-pushed the 20221020_generic-gzip-reader branch from 17a182d to 240c6a3 Compare October 20, 2022 19:54
Copy link
Collaborator

@cyriltovena cyriltovena left a comment

Choose a reason for hiding this comment

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

LGTM

We probably want to enforce in the future gzip. As this will get really quickly costly.

@simonswine simonswine merged commit 27bcadd into main Oct 21, 2022
@simonswine simonswine deleted the 20221020_generic-gzip-reader branch October 21, 2022 08:42
simonswine added a commit that referenced this pull request Oct 21, 2022
This is because the mapping ID is set to 0 on the locations.

It also adds an uncompressed profile that should be solved by #338.
simonswine added a commit to simonswine/pyroscope that referenced this pull request Jun 30, 2023
…zip-reader

Refactor gzip reading/writing
simonswine added a commit to simonswine/pyroscope that referenced this pull request Jun 30, 2023
This is because the mapping ID is set to 0 on the locations.

It also adds an uncompressed profile that should be solved by grafana/phlare#338.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
kind/enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants