Skip to content

Conversation

@dhermes
Copy link
Contributor

@dhermes dhermes commented Nov 6, 2016

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Nov 6, 2016
@daspecster
Copy link
Contributor

This LGTM but I imagine you'll want @tseaver to look at it as well.

Copy link
Contributor

@tseaver tseaver left a comment

Choose a reason for hiding this comment

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

I'd love to sea a rational analysis on the believed/claimed cost savings for the .0 bytes vs. all the induced cache-miss stuff on both client and server side. I would certainly be vetoing such pseudo-optimizations without real evidence if it were up to me.

@dhermes dhermes merged commit 9c68fdf into googleapis:master Nov 7, 2016
@dhermes dhermes deleted the nanos-tweak branch November 7, 2016 18:56
@dhermes
Copy link
Contributor Author

dhermes commented Nov 7, 2016

I doubt there is a single person or team responsible.

richkadel pushed a commit to richkadel/google-cloud-python that referenced this pull request May 6, 2017
vchudnov-g pushed a commit that referenced this pull request Sep 20, 2023
…… [(#2696)](GoogleCloudPlatform/python-docs-samples#2696)

* dialogflow: use unique names for all resources for tests to avoid collision

* lint: line length

* break up knowledge base tests and document tests from one file into individual tests

* lint

* bump library version

* ping googleapis-common-protos to avoid release breakage

* Update requirements.txt

Co-authored-by: Gus Class <[email protected]>
parthea pushed a commit that referenced this pull request Nov 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: core cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants