Skip to content

Commit

Permalink
Add exporter data classes for experimental profiling signal type.
Browse files Browse the repository at this point in the history
Changes addressing code review comments.
  • Loading branch information
jhalliday committed May 17, 2024
1 parent bda144f commit 55b8ab6
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -26,9 +26,6 @@
*/
public enum AggregationTemporality {

Check warning on line 27 in exporters/otlp/profiles/src/main/java/io/opentelemetry/exporter/otlp/profiles/AggregationTemporality.java

View check run for this annotation

Codecov / codecov/patch

exporters/otlp/profiles/src/main/java/io/opentelemetry/exporter/otlp/profiles/AggregationTemporality.java#L27

Added line #L27 was not covered by tests

/** The default AggregationTemporality, it MUST not be used. */
UNSPECIFIED,

/**
* DELTA is an AggregationTemporality for a profiler which reports changes since last report time.
*/
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,9 +5,6 @@

package io.opentelemetry.exporter.otlp.profiles;

import io.opentelemetry.api.internal.OtelEncodingUtils;
import io.opentelemetry.api.trace.SpanId;
import io.opentelemetry.api.trace.TraceId;
import javax.annotation.concurrent.Immutable;

/**
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@

package io.opentelemetry.exporter.otlp.profiles;

import javax.annotation.Nullable;
import javax.annotation.concurrent.Immutable;

/**
Expand All @@ -21,5 +22,6 @@ public interface ValueTypeData {
/** Index into string table. */
long unit();

@Nullable
AggregationTemporality aggregationTemporality();
}

0 comments on commit 55b8ab6

Please sign in to comment.