diff options
author | Mayur Kale <mayurkale@google.com> | 2018-10-17 19:25:19 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-10-17 19:25:19 -0700 |
commit | 2f53b977da6b10b7f8e8c29d83f0478d470b229c (patch) | |
tree | 2bce5e8501d36d536ae5078406a7cbf83fb1b198 /api/src/main/java | |
parent | 2b11b167c8d7633acaacc0c6aca6b829e8f4df6d (diff) | |
download | opencensus-java-2f53b977da6b10b7f8e8c29d83f0478d470b229c.tar.gz |
Remove Old Gauge and correct exception messages (#1502)
Diffstat (limited to 'api/src/main/java')
5 files changed, 13 insertions, 13 deletions
diff --git a/api/src/main/java/io/opencensus/metrics/DoubleGauge.java b/api/src/main/java/io/opencensus/metrics/DoubleGauge.java index 62730a06..de2b053b 100644 --- a/api/src/main/java/io/opencensus/metrics/DoubleGauge.java +++ b/api/src/main/java/io/opencensus/metrics/DoubleGauge.java @@ -172,15 +172,15 @@ public abstract class DoubleGauge { Utils.checkNotNull(name, "name"); Utils.checkNotNull(description, "description"); Utils.checkNotNull(unit, "unit"); - Utils.checkNotNull(labelKeys, "labelKeys should not be null."); - Utils.checkListElementNotNull(labelKeys, "labelKeys element should not be null."); + Utils.checkListElementNotNull( + Utils.checkNotNull(labelKeys, "labelKeys"), "labelKey element should not be null."); labelKeysSize = labelKeys.size(); } @Override public NoopDoublePoint getOrCreateTimeSeries(List<LabelValue> labelValues) { - Utils.checkNotNull(labelValues, "labelValues should not be null."); - Utils.checkListElementNotNull(labelValues, "labelValues element should not be null."); + Utils.checkListElementNotNull( + Utils.checkNotNull(labelValues, "labelValues"), "labelValue element should not be null."); Utils.checkArgument(labelKeysSize == labelValues.size(), "Incorrect number of labels."); return NoopDoublePoint.INSTANCE; } @@ -192,7 +192,7 @@ public abstract class DoubleGauge { @Override public void removeTimeSeries(List<LabelValue> labelValues) { - Utils.checkNotNull(labelValues, "labelValues should not be null."); + Utils.checkNotNull(labelValues, "labelValues"); } @Override diff --git a/api/src/main/java/io/opencensus/metrics/LongGauge.java b/api/src/main/java/io/opencensus/metrics/LongGauge.java index f473a8ac..8ca760be 100644 --- a/api/src/main/java/io/opencensus/metrics/LongGauge.java +++ b/api/src/main/java/io/opencensus/metrics/LongGauge.java @@ -171,8 +171,8 @@ public abstract class LongGauge { @Override public NoopLongPoint getOrCreateTimeSeries(List<LabelValue> labelValues) { - Utils.checkNotNull(labelValues, "labelValues should not be null."); - Utils.checkListElementNotNull(labelValues, "labelValues element should not be null."); + Utils.checkListElementNotNull( + Utils.checkNotNull(labelValues, "labelValues"), "labelValue element should not be null."); Utils.checkArgument(labelKeysSize == labelValues.size(), "Incorrect number of labels."); return NoopLongPoint.INSTANCE; } @@ -184,7 +184,7 @@ public abstract class LongGauge { @Override public void removeTimeSeries(List<LabelValue> labelValues) { - Utils.checkNotNull(labelValues, "labelValues should not be null."); + Utils.checkNotNull(labelValues, "labelValues"); } @Override diff --git a/api/src/main/java/io/opencensus/stats/Aggregation.java b/api/src/main/java/io/opencensus/stats/Aggregation.java index f5efed9b..9c95e847 100644 --- a/api/src/main/java/io/opencensus/stats/Aggregation.java +++ b/api/src/main/java/io/opencensus/stats/Aggregation.java @@ -180,7 +180,7 @@ public abstract class Aggregation { * @since 0.8 */ public static Distribution create(BucketBoundaries bucketBoundaries) { - Utils.checkNotNull(bucketBoundaries, "bucketBoundaries should not be null."); + Utils.checkNotNull(bucketBoundaries, "bucketBoundaries"); return new AutoValue_Aggregation_Distribution(bucketBoundaries); } diff --git a/api/src/main/java/io/opencensus/stats/AggregationData.java b/api/src/main/java/io/opencensus/stats/AggregationData.java index eb7d40b3..c6e12b67 100644 --- a/api/src/main/java/io/opencensus/stats/AggregationData.java +++ b/api/src/main/java/io/opencensus/stats/AggregationData.java @@ -288,15 +288,15 @@ public abstract class AggregationData { Utils.checkArgument(min <= max, "max should be greater or equal to min."); } - Utils.checkNotNull(bucketCounts, "bucket counts should not be null."); + Utils.checkNotNull(bucketCounts, "bucketCounts"); List<Long> bucketCountsCopy = Collections.unmodifiableList(new ArrayList<Long>(bucketCounts)); for (Long bucket : bucketCountsCopy) { - Utils.checkNotNull(bucket, "bucket should not be null."); + Utils.checkNotNull(bucket, "bucket"); } Utils.checkNotNull(exemplars, "exemplar list should not be null."); for (Exemplar exemplar : exemplars) { - Utils.checkNotNull(exemplar, "exemplar should not be null."); + Utils.checkNotNull(exemplar, "exemplar"); } return new AutoValue_AggregationData_DistributionData( diff --git a/api/src/main/java/io/opencensus/stats/BucketBoundaries.java b/api/src/main/java/io/opencensus/stats/BucketBoundaries.java index 573a9e10..61e21e6c 100644 --- a/api/src/main/java/io/opencensus/stats/BucketBoundaries.java +++ b/api/src/main/java/io/opencensus/stats/BucketBoundaries.java @@ -42,7 +42,7 @@ public abstract class BucketBoundaries { * @since 0.8 */ public static final BucketBoundaries create(List<Double> bucketBoundaries) { - Utils.checkNotNull(bucketBoundaries, "bucketBoundaries list should not be null."); + Utils.checkNotNull(bucketBoundaries, "bucketBoundaries"); List<Double> bucketBoundariesCopy = new ArrayList<Double>(bucketBoundaries); // Deep copy. // Check if sorted. if (bucketBoundariesCopy.size() > 1) { |