Skip to content

Commit

Permalink
profiles: Make mapping in Profile optional
Browse files Browse the repository at this point in the history
As commented in [0] and discussed in the OTel Profiling SIG meeting, there are
situations where a main binary for a Profile can not be identified.
For these cases mark the field optional.

[0]: #534 (comment)

Signed-off-by: Florian Lehner <florian.lehner@elastic.co>
  • Loading branch information
florianl committed Sep 26, 2024
1 parent d785e67 commit dc4eded
Showing 1 changed file with 2 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -79,6 +79,8 @@ message Profile {
repeated Sample sample = 2;
// Mapping from address ranges to the image/binary/library mapped
// into that address range. mapping[0] will be the main binary.
// If multiple binaries contribute to the Profile and no main
// binary can be identified, mapping is optional.
repeated Mapping mapping = 3;
// Locations referenced by samples via location_indices.
repeated Location location = 4;
Expand Down

0 comments on commit dc4eded

Please sign in to comment.