Skip to content

fix: Remove singleton access to request profiler. Use instance of hook instead.#2567

Merged
GaoleMeng merged 11 commits intogoogleapis:mainfrom
GaoleMeng:profiler_simplify
Jul 18, 2024
Merged

fix: Remove singleton access to request profiler. Use instance of hook instead.#2567
GaoleMeng merged 11 commits intogoogleapis:mainfrom
GaoleMeng:profiler_simplify

Conversation

@GaoleMeng
Copy link
Contributor

@GaoleMeng GaoleMeng commented Jul 17, 2024

So that we could fine grained control which stream writer to collect information

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.

@GaoleMeng GaoleMeng requested a review from a team as a code owner July 17, 2024 23:36
@GaoleMeng GaoleMeng requested a review from hongalex July 17, 2024 23:36
@product-auto-label product-auto-label bot added size: l Pull request size is large. api: bigquerystorage Issues related to the googleapis/java-bigquerystorage API. labels Jul 17, 2024
@GaoleMeng GaoleMeng force-pushed the profiler_simplify branch 5 times, most recently from db0b27b to 320492b Compare July 18, 2024 01:15
@GaoleMeng GaoleMeng force-pushed the profiler_simplify branch from 320492b to 0ac8f59 Compare July 18, 2024 18:22
@GaoleMeng GaoleMeng force-pushed the profiler_simplify branch from 0ac8f59 to baedafc Compare July 18, 2024 18:45
@GaoleMeng GaoleMeng merged commit 2f45fa2 into googleapis:main Jul 18, 2024
yifatgortler pushed a commit to yifatgortler/java-bigquerystorage that referenced this pull request Jul 25, 2024
…k instead. (googleapis#2567)

* Add profiler for request execution details. The usage of the new API
will be added in the next PR

* Add profiler for request execution details. The usage of the new API
will be added in the next PR

* add new code change

* .

* Remove directly access to singleton
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: bigquerystorage Issues related to the googleapis/java-bigquerystorage API. size: l Pull request size is large.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants