Skip to content

Commit c038580

Browse files
committed
Use pre-defined aggregation from OpenCensus for bytes histogram
1 parent 126814a commit c038580

File tree

1 file changed

+2
-23
lines changed

1 file changed

+2
-23
lines changed

census/src/main/java/io/grpc/census/internal/ObservabilityCensusConstants.java

Lines changed: 2 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -25,38 +25,17 @@
2525
import static io.opencensus.contrib.grpc.metrics.RpcMeasureConstants.GRPC_SERVER_SENT_BYTES_PER_RPC;
2626
import static io.opencensus.contrib.grpc.metrics.RpcMeasureConstants.GRPC_SERVER_STATUS;
2727

28+
import io.opencensus.contrib.grpc.metrics.RpcViewConstants;
2829
import io.opencensus.stats.Aggregation;
29-
import io.opencensus.stats.Aggregation.Distribution;
30-
import io.opencensus.stats.BucketBoundaries;
3130
import io.opencensus.stats.View;
3231
import java.util.Arrays;
33-
import java.util.Collections;
34-
import java.util.List;
3532

3633
/** Temporary holder class for the observability specific OpenCensus constants.
3734
* The class will be removed once the new views are added in OpenCensus library. */
3835
public final class ObservabilityCensusConstants {
3936

40-
static final List<Double> RPC_BYTES_BUCKET_BOUNDARIES =
41-
Collections.unmodifiableList(
42-
Arrays.asList(
43-
0.0,
44-
1024.0,
45-
2048.0,
46-
4096.0,
47-
16384.0,
48-
65536.0,
49-
262144.0,
50-
1048576.0,
51-
4194304.0,
52-
16777216.0,
53-
67108864.0,
54-
268435456.0,
55-
1073741824.0,
56-
4294967296.0));
57-
5837
static final Aggregation AGGREGATION_WITH_BYTES_HISTOGRAM =
59-
Distribution.create(BucketBoundaries.create(RPC_BYTES_BUCKET_BOUNDARIES));
38+
RpcViewConstants.GRPC_CLIENT_SENT_BYTES_PER_RPC_VIEW.getAggregation();
6039

6140
public static final View GRPC_CLIENT_SENT_COMPRESSED_MESSAGE_BYTES_PER_RPC =
6241
View.create(

0 commit comments

Comments
 (0)