docs(bigtable): Add todos

bhshkh · Dec 12, 2024 · cf414dd · cf414dd
1 parent 46fc993
commit cf414dd
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/bigtable/bigtable.go b/bigtable/bigtable.go
@@ -126,10 +126,10 @@ func NewClientWithConfig(ctx context.Context, project, instance string, config C
 	o = append(o, internaloption.AllowNonDefaultServiceAccount(true))
 	o = append(o, opts...)
 
+	// TODO(b/372244283): Remove after b/358175516 has been fixed
 	asyncRefreshMetricAttrs := metricsTracerFactory.clientAttributes
 	asyncRefreshMetricAttrs = append(asyncRefreshMetricAttrs,
 		attribute.String(metricLabelKeyTag, "async_refresh_dry_run"),
-
 		// Table, cluster and zone are unknown at this point
 		// Use default values
 		attribute.String(monitoredResLabelKeyTable, defaultTable),
@@ -140,9 +140,10 @@ func NewClientWithConfig(ctx context.Context, project, instance string, config C
 		metricsTracerFactory.debugTags.Add(context.Background(), 1,
 			metric.WithAttributes(asyncRefreshMetricAttrs...))
 	}))
+
 	connPool, err := gtransport.DialPool(ctx, o...)
 	if err != nil {
-		return nil, fmt.Errorf("dialing: %w", err)
+		return nil, err
 	}
 
 	return &Client{