8000 Use Futures.transform instead of lazyTransform when possible by mziccard · Pull Request #1168 · googleapis/google-cloud-java · GitHub
[go: up one dir, main page]

Skip to content

Conversation

@mziccard
Copy link
Contributor

This fixes #1148

@mattnworb this should make you happy :) Any change you can give it a try?

@mziccard mziccard added api: pubsub Issues related to the Pub/Sub API. api: logging Issues related to the Cloud Logging API. labels Aug 17, 2016
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Aug 17, 2016
@mattnworb
Copy link

Awesome, thanks so much @mziccard !

I built a 0.2.8-SNAPSHOT from your branch and ran it in one of our applications that calls pubsub.publishAsync(). After executing the code path that calls that method a bunch of times, I do not see any threads created via the ThreadFactory that JdkFutureAdapters uses (a thread named like ListenableFutureAdapter-thread-%d). When running with prior versions, I see a few threads like ListenableFutureAdapter-thread-1.

thanks again!

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.006%) to 85.864% when pulling 5e99284 on mziccard:future-transform into 36ba7c3 on GoogleCloudPlatform:master.

8000

@mziccard
Copy link
Contributor Author

/cc @lesv

@lesv
Copy link
Contributor
lesv commented Aug 18, 2016

LGTM - I'm presuming some integration tests will run to verify this for logging.

@mziccard mziccard merged commit 4b1f0e9 into googleapis:master Aug 21, 2016
github-actions bot pushed a commit that referenced this pull request Jun 23, 2022
- [ ] Regenerate this pull request now.

PiperOrigin-RevId: 456641589

Source-Link: googleapis/googleapis@8a251f5

Source-Link: https://github.com/googleapis/googleapis-gen/commit/4ca52a529cf01308d9714950edffbea3560cfbdb
Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiNGNhNTJhNTI5Y2YwMTMwOGQ5NzE0OTUwZWRmZmJlYTM1NjBjZmJkYiJ9
github-actions bot pushed a commit to suztomo/google-cloud-java that referenced this pull request Jun 29, 2022
Source-Link: googleapis/synthtool@1a2878d
Post-Processor: gcr.io/repo-automation-bots/owlbot-java:latest@sha256:12cbdc4f137152a1a8b83f50ebc1a4098be0d3baff78b46efaec306f7cfafd13

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
github-actions bot pushed a commit that referenced this pull request Jul 1, 2022
Source-Link: googleapis/synthtool@1a2878d
Post-Processor: gcr.io/repo-automation-bots/owlbot-java:latest@sha256:12cbdc4f137152a1a8b83f50ebc1a4098be0d3baff78b46efaec306f7cfafd13

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
github-actions bot pushed a commit to suztomo/google-cloud-java that referenced this pull request Jul 1, 2022
🤖 I have created a release *beep* *boop*
---


## [2.3.0](googleapis/java-automl@v2.2.4...v2.3.0) (2022-06-30)


### Features

* Enable REST transport for most of Java and Go clients ([googleapis#1168](googleapis/java-automl#1168)) ([2fd0096](googleapis/java-automl@2fd0096))


### Bug Fixes

* update gapic-generator-java with mock service generation fixes ([googleapis#1174](googleapis/java-automl#1174)) ([e5fb07e](googleapis/java-automl@e5fb07e))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
meltsufin pushed a commit that referenced this pull request Dec 22, 2025
…to v3.12.0 (#1168)

* chore(deps): update dependency com.google.cloud:google-cloud-logging to v3.12.0

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: logging Issues related to the Cloud Logging API. api: pubsub Issues related to the Pub/Sub API. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

request: return ListenableFutures in PubSub async methods rather than bare Futures

5 participants

0