Skip to content

Conversation

@sushanb
Copy link
Contributor

@sushanb sushanb commented Oct 29, 2025

No description provided.

@sushanb sushanb requested a review from mutianf October 29, 2025 16:44
@sushanb sushanb requested review from a team as code owners October 29, 2025 16:44
@product-auto-label product-auto-label bot added size: m Pull request size is medium. api: bigtable Issues related to the googleapis/java-bigtable API. labels Oct 29, 2025
AttributeKey.stringKey("grpc_disconnect_error");
static final AttributeKey<String> GRPC_LB_LOCALITY_KEY =
AttributeKey.stringKey("grpc_lb_locality");
static final AttributeKey<String> GRPC_TARGET_KEY = AttributeKey.stringKey("grpc_target");
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this should be

Suggested change
static final AttributeKey<String> GRPC_TARGET_KEY = AttributeKey.stringKey("grpc_target");
static final AttributeKey<String> GRPC_TARGET_KEY = AttributeKey.stringKey("grpc.target");

?

static final AttributeKey<String> GRPC_DISCONNECT_ERROR_KEY =
AttributeKey.stringKey("grpc_disconnect_error");
static final AttributeKey<String> GRPC_LB_LOCALITY_KEY =
AttributeKey.stringKey("grpc_lb_locality");
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this should be

Suggested change
AttributeKey.stringKey("grpc_lb_locality");
AttributeKey.stringKey("grpc.lb.locality");

?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed with the format.

All these attributes go reformatting here.

@sushanb sushanb merged commit e3e6e99 into googleapis:main Oct 31, 2025
24 checks passed
@sushanb sushanb deleted the kk branch October 31, 2025 20:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: bigtable Issues related to the googleapis/java-bigtable API. size: m Pull request size is medium.

2 participants