Skip to content

Conversation

@Gautam04
Copy link
Contributor

@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
Contributor 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
Contributor Author

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

@MarkDuckworth MarkDuckworth added automerge Merge the pull request once unit tests and other checks pass. kokoro:force-run Add this label to force Kokoro to re-run the tests. labels Dec 18, 2025
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Dec 18, 2025
@gcf-merge-on-green gcf-merge-on-green bot merged commit 926fd8c into googleapis:main Dec 18, 2025
24 checks passed
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Dec 18, 2025
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.

6 participants