Skip to content

chore: add builtin observability support - #179

Merged
mayurkale22 merged 7 commits into
mainfrom
metrics
Aug 5, 2025
Merged

chore: add builtin observability support#179
mayurkale22 merged 7 commits into
mainfrom
metrics

Conversation

@mayurkale22

Copy link
Copy Markdown
Collaborator

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

  • Make sure to open an issue as a bug/issue before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
  • Ensure the tests and linter pass
  • Code coverage does not decrease (if any source code was changed)
  • Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> ☕️

If you write sample code, please follow the samples format.

@mayurkale22
mayurkale22 requested a review from ShuranZhang August 1, 2025 01:46
Comment thread google-cloud-spanner-cassandra/pom.xml
@ath-08

ath-08 commented Aug 1, 2025

Copy link
Copy Markdown
Contributor

I think we should add atleast a few basic test cases to verify metrics accuracy. That way any unintentional changes won't break the metrics values.

@mayurkale22

Copy link
Copy Markdown
Collaborator Author

I think we should add atleast a few basic test cases to verify metrics accuracy. That way any unintentional changes won't break the metrics values.

Done, PTAL

@mayurkale22
mayurkale22 requested a review from ath-08 August 4, 2025 05:24
@mayurkale22
mayurkale22 merged commit 684503b into main Aug 5, 2025
16 checks passed
@ath-08
ath-08 deleted the metrics branch September 9, 2025 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants