Skip to content

Conversation

@Gautam04
Copy link

@Gautam04 Gautam04 commented Nov 30, 2025

Adding a new span attribute called gcp.resource.name which contains an identifier to a particular firestore database in the following format:

//firestore.googleapis.com/projects/{project}/databases/{database_id}

Example:

//firestore.googleapis.com/projects/my_project/databases/(default)

@Gautam04 Gautam04 requested a review from a team as a code owner November 30, 2025 15:31
@product-auto-label product-auto-label bot added size: xs Pull request size is extra small. api: firestore Issues related to the googleapis/java-firestore API. labels Nov 30, 2025
@dconeybe
Copy link
Contributor

dconeybe commented Dec 1, 2025

@Gautam04 Did you have someone in mind to review this PR?

@dconeybe
Copy link
Contributor

dconeybe commented Dec 1, 2025

Googlers see b/437960147 and g/firestore-sdk-team/c/uvdOjkrQ2Ns/m/1wJrFyFdAwAJ for some background details

@Gautam04
Copy link
Author

Gautam04 commented Dec 5, 2025

@Gautam04 Did you have someone in mind to review this PR?

No,
I am looking for someone to review this PR actually.

@dconeybe
Copy link
Contributor

dconeybe commented Dec 5, 2025

@Gautam04 Did you have someone in mind to review this PR?

No, I am looking for someone to review this PR actually.

I've added @wu-hui and @MarkDuckworth as reviewers, this this PR and googleapis/nodejs-firestore#2454

Copy link
Contributor

@MarkDuckworth MarkDuckworth left a comment

Choose a reason for hiding this comment

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

LGTM

@Gautam04
Copy link
Author

@wu-hui Could you please check this PR?
Thanks!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: firestore Issues related to the googleapis/java-firestore API. size: xs Pull request size is extra small.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants