chore: remove stale metrics code

Change-Id: Ib98c15b0531f70d1c0bd65ffbd2161b3e605448a

Author: igorbernstein2

google-cloud-bigtable/src/main/java/com/google/cloud/bigtable/data/v2/internal/csm/MetricRegistry.java

@@ -53,7 +53,7 @@
  * </ul>
  */
 public class MetricRegistry {
-  static final String METER_NAME = "bigtable.googleapis.com/internal/client/";
+  public static final String METER_NAME = "bigtable.googleapis.com/internal/client/";
 
   final TableOperationLatency operationLatencyMetric;
   final TableAttemptLatency attemptLatencyMetric;

google-cloud-bigtable/src/main/java/com/google/cloud/bigtable/data/v2/internal/csm/MetricsImpl.java

@@ -25,7 +25,6 @@
 import com.google.cloud.bigtable.data.v2.internal.csm.attributes.ClientInfo;
 import com.google.cloud.bigtable.data.v2.internal.csm.attributes.EnvInfo;
 import com.google.cloud.bigtable.data.v2.stub.metrics.BigtableCloudMonitoringExporter;
-import com.google.cloud.bigtable.data.v2.stub.metrics.BuiltinMetricsConstants;
 import com.google.cloud.bigtable.data.v2.stub.metrics.BuiltinMetricsTracerFactory;
 import com.google.cloud.bigtable.data.v2.stub.metrics.ChannelPoolMetricsTracer;
 import com.google.cloud.bigtable.data.v2.stub.metrics.CompositeTracerFactory;
@@ -42,18 +41,15 @@
 import io.opencensus.tags.Tagger;
 import io.opentelemetry.api.OpenTelemetry;
 import io.opentelemetry.sdk.OpenTelemetrySdk;
-import io.opentelemetry.sdk.metrics.InstrumentSelector;
 import io.opentelemetry.sdk.metrics.SdkMeterProvider;
 import io.opentelemetry.sdk.metrics.SdkMeterProviderBuilder;
-import io.opentelemetry.sdk.metrics.View;
 import io.opentelemetry.sdk.metrics.export.MetricExporter;
 import io.opentelemetry.sdk.metrics.export.PeriodicMetricReader;
 import io.opentelemetry.sdk.metrics.export.PeriodicMetricReaderBuilder;
 import java.io.Closeable;
 import java.io.IOException;
 import java.util.ArrayList;
 import java.util.List;
-import java.util.Map;
 import java.util.concurrent.ScheduledExecutorService;
 import java.util.concurrent.ScheduledFuture;
 import javax.annotation.Nullable;
@@ -100,7 +96,7 @@ public MetricsImpl(
               // Disable default grpc metrics
               .disableAllMetrics()
               // Enable specific grpc metrics
-              .enableMetrics(BuiltinMetricsConstants.GRPC_METRICS.keySet())
+              .enableMetrics(metricRegistry.getGrpcMetricNames())
               .build();
     } else {
       this.grpcOtel = null;
@@ -185,19 +181,14 @@ public static OpenTelemetrySdk createBuiltinOtel(
 
     SdkMeterProviderBuilder meterProvider = SdkMeterProvider.builder();
 
-    for (Map.Entry<InstrumentSelector, View> entry :
-        BuiltinMetricsConstants.getAllViews().entrySet()) {
-      meterProvider.registerView(entry.getKey(), entry.getValue());
-    }
-
-    for (Map.Entry<InstrumentSelector, View> e :
-        BuiltinMetricsConstants.getInternalViews().entrySet()) {
-      meterProvider.registerView(e.getKey(), e.getValue());
-    }
-
     MetricExporter publicExporter =
         BigtableCloudMonitoringExporter.create(
-            metricRegistry, EnvInfo::detect, clientInfo, credentials, metricsEndpoint, universeDomain);
+            metricRegistry,
+            EnvInfo::detect,
+            clientInfo,
+            credentials,
+            metricsEndpoint,
+            universeDomain);
     PeriodicMetricReaderBuilder readerBuilder =
         PeriodicMetricReader.builder(publicExporter).setExecutor(executor);
     meterProvider.registerMetricReader(readerBuilder.build());

google-cloud-bigtable/src/main/java/com/google/cloud/bigtable/data/v2/internal/csm/metrics/ClientBatchWriteFlowControlFactor.java

@@ -25,7 +25,7 @@
 import io.opentelemetry.api.metrics.Meter;
 
 public class ClientBatchWriteFlowControlFactor extends MetricWrapper<ClientSchema> {
-  private static final String NAME =
+  public static final String NAME =
       "bigtable.googleapis.com/internal/client/batch_write_flow_control_factor";
 
   public ClientBatchWriteFlowControlFactor() {

google-cloud-bigtable/src/main/java/com/google/cloud/bigtable/data/v2/internal/csm/metrics/ClientBatchWriteFlowControlTargetQps.java

@@ -24,7 +24,7 @@
 import io.opentelemetry.api.metrics.Meter;
 
 public class ClientBatchWriteFlowControlTargetQps extends MetricWrapper<ClientSchema> {
-  private static final String NAME =
+  public static final String NAME =
       "bigtable.googleapis.com/internal/client/batch_write_flow_control_target_qps";
 
   public ClientBatchWriteFlowControlTargetQps() {

google-cloud-bigtable/src/main/java/com/google/cloud/bigtable/data/v2/internal/csm/metrics/ClientChannelPoolOutstandingRpcs.java

@@ -28,7 +28,7 @@
 import java.util.stream.Collectors;
 
 public class ClientChannelPoolOutstandingRpcs extends MetricWrapper<ClientSchema> {
-  private static final String NAME =
+  public static final String NAME =
       "bigtable.googleapis.com/internal/client/connection_pool/outstanding_rpcs";
 
   private static final List<Long> BUCKETS =

google-cloud-bigtable/src/main/java/com/google/cloud/bigtable/data/v2/internal/csm/metrics/ClientPerConnectionErrorCount.java

@@ -32,7 +32,7 @@
 import java.util.Set;
 
 public class ClientPerConnectionErrorCount extends MetricWrapper<ClientSchema> {
-  private static final String NAME =
+  public static final String NAME =
       "bigtable.googleapis.com/internal/client/per_connection_error_count";
 
   static final List<Long> BUCKETS =

google-cloud-bigtable/src/main/java/com/google/cloud/bigtable/data/v2/internal/csm/metrics/Constants.java

@@ -54,7 +54,7 @@ private MetricLabels() {}
         AttributeKey.stringKey("app_profile");
     public static final AttributeKey<String> DEBUG_TAG_KEY = AttributeKey.stringKey("tag");
 
-    static final AttributeKey<Boolean> APPLIED_KEY = AttributeKey.booleanKey("applied");
+    public static final AttributeKey<Boolean> APPLIED_KEY = AttributeKey.booleanKey("applied");
 
     static final AttributeKey<String> CHANNEL_POOL_LB_POLICY = AttributeKey.stringKey("lb_policy");
     static final AttributeKey<String> DP_REASON_KEY = AttributeKey.stringKey("reason");

google-cloud-bigtable/src/main/java/com/google/cloud/bigtable/data/v2/internal/csm/metrics/TableApplicationBlockingLatency.java

@@ -32,8 +32,7 @@
 import javax.annotation.Nullable;
 
 public class TableApplicationBlockingLatency extends MetricWrapper<TableSchema> {
-  private static final String NAME =
-      "bigtable.googleapis.com/internal/client/application_latencies";
+  public static final String NAME = "bigtable.googleapis.com/internal/client/application_latencies";
 
   public TableApplicationBlockingLatency() {
     super(TableSchema.INSTANCE, NAME);

google-cloud-bigtable/src/main/java/com/google/cloud/bigtable/data/v2/internal/csm/metrics/TableAttemptLatency.java

@@ -33,7 +33,7 @@
 import javax.annotation.Nullable;
 
 public class TableAttemptLatency extends MetricWrapper<TableSchema> {
-  private static final String NAME = "bigtable.googleapis.com/internal/client/attempt_latencies";
+  public static final String NAME = "bigtable.googleapis.com/internal/client/attempt_latencies";
 
   public TableAttemptLatency() {
     super(TableSchema.INSTANCE, NAME);

google-cloud-bigtable/src/main/java/com/google/cloud/bigtable/data/v2/internal/csm/metrics/TableAttemptLatency2.java

@@ -34,7 +34,7 @@
 import javax.annotation.Nullable;
 
 public class TableAttemptLatency2 extends MetricWrapper<TableSchema> {
-  private static final String NAME = "bigtable.googleapis.com/internal/client/attempt_latencies2";
+  public static final String NAME = "bigtable.googleapis.com/internal/client/attempt_latencies2";
 
   public TableAttemptLatency2() {
     super(TableSchema.INSTANCE, NAME);