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

Use Futures.transform instead of lazyTransform when possible #1168

Merged
merged 1 commit into from Aug 21, 2016

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.

@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: googleapis/googleapis-gen@4ca52a5
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).
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