docs(bigtable): Add todos (#11280)

bhshkh · web-flow · commit d4f2449c5c21 · 2024-12-16T19:06:40.000-05:00
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{
