Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(firestore): add opencensus tracing support #2942

Merged
merged 8 commits into from Feb 4, 2021

Conversation

@AlisskaPie
Copy link
Contributor

@AlisskaPie AlisskaPie commented Sep 30, 2020

No description provided.

@google-cla google-cla bot added the cla: yes label Sep 30, 2020
@AlisskaPie AlisskaPie marked this pull request as ready for review Sep 30, 2020
@AlisskaPie AlisskaPie requested a review from tritone as a code owner Sep 30, 2020
@AlisskaPie AlisskaPie requested a review from codyoss Sep 30, 2020
Copy link
Collaborator

@tritone tritone left a comment

Generally looks good to me. @BenWhitehead could you take a look?

Loading

firestore/transaction.go Show resolved Hide resolved
Loading
firestore/transaction.go Show resolved Hide resolved
Loading
firestore/client.go Outdated Show resolved Hide resolved
Loading
@crwilcox crwilcox requested a review from as a code owner Dec 2, 2020
Copy link
Contributor Author

@AlisskaPie AlisskaPie left a comment

@crwilcox Thank you!

Loading

firestore/transaction.go Show resolved Hide resolved
Loading
firestore/client.go Outdated Show resolved Hide resolved
Loading
@tritone
Copy link
Collaborator

@tritone tritone commented Jan 23, 2021

@crwilcox could you take another look at this?

Loading

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

6 participants