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

Speed up ResourceManager tests on Travis #504

Closed
ajkannan opened this issue Dec 22, 2015 · 3 comments
Closed

Speed up ResourceManager tests on Travis #504

ajkannan opened this issue Dec 22, 2015 · 3 comments
Assignees
Labels
api: cloudresourcemanager Issues related to the Resource Manager API.

Comments

@ajkannan
Copy link

Tests that use the mock resource manager helper take a relatively long time to complete on Travis. LocalResourceManagerHelperTest takes ~100 seconds and ResourceManagerImplTest takes ~45 seconds. When running locally, these tests only tend to take 3-5 seconds. This is likely because the HttpServer library used for the mock spins up threads and also uses sockets, and Travis probably handles these operations more slowly. We should think of a way to speed up these tests.

@ajkannan ajkannan added api: cloudresourcemanager Issues related to the Resource Manager API. build labels Dec 22, 2015
@ajkannan ajkannan self-assigned this Dec 22, 2015
@mziccard
Copy link
Contributor

Not sure if this is a pattern but the two tests seem to take much less in oraclejdk8 builds:

  • LocalResourceManagerHelperTest ~ 5 sec
  • ResourceManagerImplTest ~ 3 sec

@ajkannan
Copy link
Author

Interesting, the faster test runs in version 8 build seem to be a trend. Perhaps related, ConcurrentHashMap (used by the mock) seems to have radically changed in implementation between version 7 and 8. There must be a lot of lock contention if this is indeed the cause.

@aozarov
Copy link
Contributor

aozarov commented Dec 23, 2015

see this.

A suggested workaround to add exchange.getResponseHeaders().add("Connection", "close");
just before exchange.sendResponseHeaders (line 189) seems to make it much faster.

github-actions bot pushed a commit to suztomo/google-cloud-java that referenced this issue Jun 29, 2022
This PR was generated using Autosynth. 🌈


<details><summary>Log from Synthtool</summary>

```
2021-05-26 21:01:26,717 synthtool [DEBUG] > Executing /root/.cache/synthtool/java-resourcemanager/.github/readme/synth.py.
On branch autosynth-readme
nothing to commit, working tree clean
2021-05-26 21:01:27,926 synthtool [DEBUG] > Wrote metadata to .github/readme/synth.metadata/synth.metadata.

```
</details>

Full log will be available here:
https://source.cloud.google.com/results/invocations/7c08b2db-72c5-443c-aa3e-55d4ca103164/targets

- [ ] To automatically regenerate this PR, check this box. (May take up to 24 hours.)
github-actions bot pushed a commit that referenced this issue Jun 30, 2022
Source-Link: googleapis/synthtool@7a220e2
Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-java:latest@sha256:6d4e3a15c62cfdcb823d60e16da7521e7c6fc00eba07c8ff12e4de9924a57d28
github-actions bot pushed a commit to suztomo/google-cloud-java that referenced this issue Jul 1, 2022
🤖 I have created a release *beep* *boop*
---


### Updating meta-information for bleeding-edge SNAPSHOT release.

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
github-actions bot pushed a commit that referenced this issue Jul 20, 2022
…plugin to v3 (#504)

[![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [org.apache.maven.plugins:maven-deploy-plugin](https://maven.apache.org/plugins/) | `2.8.2` -> `3.0.0` | [![age](https://badges.renovateapi.com/packages/maven/org.apache.maven.plugins:maven-deploy-plugin/3.0.0/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/maven/org.apache.maven.plugins:maven-deploy-plugin/3.0.0/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/maven/org.apache.maven.plugins:maven-deploy-plugin/3.0.0/compatibility-slim/2.8.2)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/maven/org.apache.maven.plugins:maven-deploy-plugin/3.0.0/confidence-slim/2.8.2)](https://docs.renovatebot.com/merge-confidence/) |

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, click this checkbox.

---

This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/java-os-config).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzMi4xMTcuNCIsInVwZGF0ZWRJblZlciI6IjMyLjExNy40In0=-->
github-actions bot pushed a commit that referenced this issue Aug 9, 2022
…0.0 (#504)

[![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [com.google.cloud:google-cloud-storage](https://togithub.com/googleapis/java-storage) | `2.8.0` -> `2.10.0` | [![age](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-storage/2.10.0/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-storage/2.10.0/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-storage/2.10.0/compatibility-slim/2.8.0)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-storage/2.10.0/confidence-slim/2.8.0)](https://docs.renovatebot.com/merge-confidence/) |

---

### Release Notes

<details>
<summary>googleapis/java-storage</summary>

### [`v2.10.0`](https://togithub.com/googleapis/java-storage/blob/HEAD/CHANGELOG.md#&#8203;2100-httpsgithubcomgoogleapisjava-storagecomparev293v2100-2022-07-13)

[Compare Source](https://togithub.com/googleapis/java-storage/compare/v2.9.3...v2.10.0)

##### Features

-   Custom Placement Config Dual Region Support ([#&#8203;1470](https://togithub.com/googleapis/java-storage/issues/1470)) ([36440fc](https://togithub.com/googleapis/java-storage/commit/36440fc587976acc78f4dae480214b80ad3fc477))

##### Bug Fixes

-   enable longpaths support for windows test ([#&#8203;1485](https://togithub.com/googleapis/java-storage/issues/1485)) ([#&#8203;1506](https://togithub.com/googleapis/java-storage/issues/1506)) ([e36e809](https://togithub.com/googleapis/java-storage/commit/e36e80934644ccd5e873754c625367f24e5fc7f3))

##### Documentation

-   **sample:** upgrade maven-surefire and remove junit-platform-native ([#&#8203;1476](https://togithub.com/googleapis/java-storage/issues/1476)) ([9b222e4](https://togithub.com/googleapis/java-storage/commit/9b222e483937bd0639123899442c1e863683718a))

##### Dependencies

-   update dependency org.graalvm.buildtools:native-maven-plugin to v0.9.13 ([#&#8203;1501](https://togithub.com/googleapis/java-storage/issues/1501)) ([36e3898](https://togithub.com/googleapis/java-storage/commit/36e389857ea3f86350af5763c829e5f1caaaa3ec))

### [`v2.9.3`](https://togithub.com/googleapis/java-storage/blob/HEAD/CHANGELOG.md#&#8203;293-httpsgithubcomgoogleapisjava-storagecomparev292v293-2022-07-08)

[Compare Source](https://togithub.com/googleapis/java-storage/compare/v2.9.2...v2.9.3)

##### Dependencies

-   update dependency com.google.apis:google-api-services-storage to v1-rev20220705-1.32.1 ([#&#8203;1494](https://togithub.com/googleapis/java-storage/issues/1494)) ([5612f0f](https://togithub.com/googleapis/java-storage/commit/5612f0f929582c9e720d67683f9441d96080fb14))

### [`v2.9.2`](https://togithub.com/googleapis/java-storage/blob/HEAD/CHANGELOG.md#&#8203;292-httpsgithubcomgoogleapisjava-storagecomparev291v292-2022-07-06)

[Compare Source](https://togithub.com/googleapis/java-storage/compare/v2.9.1...v2.9.2)

##### Dependencies

-   update kms.version to v0.97.0 ([#&#8203;1488](https://togithub.com/googleapis/java-storage/issues/1488)) ([852cafe](https://togithub.com/googleapis/java-storage/commit/852cafe7db018a6fd666c3490c96eeca34c4a1ed))

### [`v2.9.1`](https://togithub.com/googleapis/java-storage/blob/HEAD/CHANGELOG.md#&#8203;291-httpsgithubcomgoogleapisjava-storagecomparev290v291-2022-07-06)

[Compare Source](https://togithub.com/googleapis/java-storage/compare/v2.9.0...v2.9.1)

##### Dependencies

-   update dependency com.google.cloud:google-cloud-pubsub to v1.120.0 ([#&#8203;1487](https://togithub.com/googleapis/java-storage/issues/1487)) ([5a8f6e5](https://togithub.com/googleapis/java-storage/commit/5a8f6e5f29e4dfb2dc03cad8c77e4b504bc83019))

### [`v2.9.0`](https://togithub.com/googleapis/java-storage/blob/HEAD/CHANGELOG.md#&#8203;290-httpsgithubcomgoogleapisjava-storagecomparev281v290-2022-06-27)

[Compare Source](https://togithub.com/googleapis/java-storage/compare/v2.8.1...v2.9.0)

##### Features

-   Enable REST transport for most of Java and Go clients ([#&#8203;1469](https://togithub.com/googleapis/java-storage/issues/1469)) ([cfb6fd8](https://togithub.com/googleapis/java-storage/commit/cfb6fd8c1f13cb2115e5ff7ea7af41a9e27986b1))

##### Documentation

-   Clarified how clients should work with resumable uploads ([#&#8203;1457](https://togithub.com/googleapis/java-storage/issues/1457)) ([15baa27](https://togithub.com/googleapis/java-storage/commit/15baa27c97aacca115981f5c14d942a8ddd6cb1f))

##### Dependencies

-   update dependency com.google.cloud:google-cloud-shared-dependencies to v2.13.0 ([#&#8203;1468](https://togithub.com/googleapis/java-storage/issues/1468)) ([1a817fe](https://togithub.com/googleapis/java-storage/commit/1a817fef87e626e74dab47eaff33c1d9d3c56bb5))
-   update dependency org.graalvm.buildtools:junit-platform-native to v0.9.12 ([#&#8203;1463](https://togithub.com/googleapis/java-storage/issues/1463)) ([d42c570](https://togithub.com/googleapis/java-storage/commit/d42c5704e4fe7341dfdd0562fa0bdbcd4bc99c7e))
-   update dependency org.graalvm.buildtools:native-maven-plugin to v0.9.12 ([#&#8203;1464](https://togithub.com/googleapis/java-storage/issues/1464)) ([2f35a4f](https://togithub.com/googleapis/java-storage/commit/2f35a4f597870a4506119946510ba82fbb7f9449))
-   update kms.version to v0.96.3 ([#&#8203;1471](https://togithub.com/googleapis/java-storage/issues/1471)) ([ceb7fa6](https://togithub.com/googleapis/java-storage/commit/ceb7fa665c35f6b00aebaa665246597b9ee4632a))

### [`v2.8.1`](https://togithub.com/googleapis/java-storage/blob/HEAD/CHANGELOG.md#&#8203;281-httpsgithubcomgoogleapisjava-storagecomparev280v281-2022-06-13)

[Compare Source](https://togithub.com/googleapis/java-storage/compare/v2.8.0...v2.8.1)

##### Dependencies

-   update dependency com.google.apis:google-api-services-storage to v1-rev20220608-1.32.1 ([#&#8203;1448](https://togithub.com/googleapis/java-storage/issues/1448)) ([96676cd](https://togithub.com/googleapis/java-storage/commit/96676cd830aca27c23c08e02e8cc7c58dece686c))

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, click this checkbox.

---

This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/java-retail).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzMi4xMzUuMSIsInVwZGF0ZWRJblZlciI6IjMyLjEzNS4xIn0=-->
github-actions bot pushed a commit that referenced this issue Aug 9, 2022
🤖 I have created a release *beep* *boop*
---


## [2.4.0](googleapis/java-retail@v2.3.0...v2.4.0) (2022-08-09)


### Features

* allow adding labels in search requests ([2be6d88](googleapis/java-retail@2be6d88))
* allow adding labels in search requests ([2be6d88](googleapis/java-retail@2be6d88))
* allow disabling spell check in search requests ([2be6d88](googleapis/java-retail@2be6d88))
* allow disabling spell check in search requests ([2be6d88](googleapis/java-retail@2be6d88))
* allow enabling recommendation filtering on custom attributes ([2be6d88](googleapis/java-retail@2be6d88))
* allow enabling recommendation filtering on custom attributes ([2be6d88](googleapis/java-retail@2be6d88))
* allow returning min/max values on search numeric facets ([2be6d88](googleapis/java-retail@2be6d88))
* allow returning min/max values on search numeric facets ([2be6d88](googleapis/java-retail@2be6d88))
* allow skiping default branch protection when doing product full import ([2be6d88](googleapis/java-retail@2be6d88))
* allow to return min/max values on search numeric facets ([2be6d88](googleapis/java-retail@2be6d88))
* allow to use serving configs as an alias of placements ([2be6d88](googleapis/java-retail@2be6d88))
* allow using serving configs as an alias of placements ([2be6d88](googleapis/java-retail@2be6d88))
* allow using serving configs as an alias of placements ([2be6d88](googleapis/java-retail@2be6d88))
* new model service to manage recommendation models ([#497](googleapis/java-retail#497)) ([2be6d88](googleapis/java-retail@2be6d88))
* return output BigQuery table on product / event export response ([2be6d88](googleapis/java-retail@2be6d88))
* return output BigQuery table on product / event export response ([2be6d88](googleapis/java-retail@2be6d88))
* support case insensitive match on search facets ([2be6d88](googleapis/java-retail@2be6d88))
* support case insensitive match on search facets ([2be6d88](googleapis/java-retail@2be6d88))
* support case insensitive match on search facets ([2be6d88](googleapis/java-retail@2be6d88))


### Bug Fixes

* enable longpaths support for windows test ([#1485](https://github.com/googleapis/java-retail/issues/1485)) ([#487](googleapis/java-retail#487)) ([ff6af57](googleapis/java-retail@ff6af57))


### Documentation

* keep the API doc up-to-date with recent changes ([2be6d88](googleapis/java-retail@2be6d88))
* keep the API doc up-to-date with recent changes ([2be6d88](googleapis/java-retail@2be6d88))
* keep the API doc up-to-date with recent changes ([2be6d88](googleapis/java-retail@2be6d88))
* **samples:** Refactoring product package (CRUD) ([#417](googleapis/java-retail#417)) ([d0c5f39](googleapis/java-retail@d0c5f39))
* **samples:** Refactoring search package ([#415](googleapis/java-retail#415)) ([f7e3b16](googleapis/java-retail@f7e3b16))


### Dependencies

* update dependency com.google.cloud:google-cloud-shared-dependencies to v3 ([#499](googleapis/java-retail#499)) ([eb0b64d](googleapis/java-retail@eb0b64d))
* update dependency com.google.cloud:google-cloud-storage to v2.10.0 ([#504](googleapis/java-retail#504)) ([38ffa21](googleapis/java-retail@38ffa21))
* update dependency com.google.cloud:google-cloud-storage to v2.11.0 ([#507](googleapis/java-retail#507)) ([fdd05c6](googleapis/java-retail@fdd05c6))
* update dependency com.google.cloud:google-cloud-storage to v2.11.1 ([#509](googleapis/java-retail#509)) ([05f2d9b](googleapis/java-retail@05f2d9b))
* update dependency com.google.cloud:google-cloud-storage to v2.11.2 ([#510](googleapis/java-retail#510)) ([6dccbdf](googleapis/java-retail@6dccbdf))
* update dependency com.google.code.gson:gson to v2.9.1 ([#500](googleapis/java-retail#500)) ([4f864e8](googleapis/java-retail@4f864e8))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
github-actions bot pushed a commit that referenced this issue Oct 4, 2022
🤖 I have created a release *beep* *boop*
---


## [2.3.5](https://togithub.com/googleapis/java-workflows/compare/v2.3.4...v2.3.5) (2022-10-03)


### Dependencies

* Update dependency certifi to v2022.9.24 ([#500](https://togithub.com/googleapis/java-workflows/issues/500)) ([6455ca3](https://togithub.com/googleapis/java-workflows/commit/6455ca3bc3d82d1db0ca928f5bc23b564e754ccc))
* Update dependency com.google.cloud:google-cloud-shared-dependencies to v3.0.4 ([#510](https://togithub.com/googleapis/java-workflows/issues/510)) ([ab6207c](https://togithub.com/googleapis/java-workflows/commit/ab6207c79a10bc0104c49e653831da873686d668))
* Update dependency gcp-releasetool to v1.8.8 ([#501](https://togithub.com/googleapis/java-workflows/issues/501)) ([764faa0](https://togithub.com/googleapis/java-workflows/commit/764faa092730a12eb7416a473ed8a9e964cc830f))
* Update dependency google-auth to v2.11.1 ([#502](https://togithub.com/googleapis/java-workflows/issues/502)) ([56bb21e](https://togithub.com/googleapis/java-workflows/commit/56bb21e4373d87d80455c9e4c354120c1b23716b))
* Update dependency googleapis-common-protos to v1.56.4 ([#504](https://togithub.com/googleapis/java-workflows/issues/504)) ([e9b4cb7](https://togithub.com/googleapis/java-workflows/commit/e9b4cb7d94c969b0a95366c46bf68fb64dcca251))
* Update dependency pyjwt to v2.5.0 ([#505](https://togithub.com/googleapis/java-workflows/issues/505)) ([e9ed50f](https://togithub.com/googleapis/java-workflows/commit/e9ed50f175445eb566c4bf9b8098d520a49057a5))
* Update dependency requests to v2.28.1 ([#506](https://togithub.com/googleapis/java-workflows/issues/506)) ([f858650](https://togithub.com/googleapis/java-workflows/commit/f85865024f2c2c18a509766465b7c987653672b8))

---
This PR was generated with [Release Please](https://togithub.com/googleapis/release-please). See [documentation](https://togithub.com/googleapis/release-please#release-please).
suztomo pushed a commit that referenced this issue Feb 1, 2023
#504)

- [ ] Regenerate this pull request now.

PiperOrigin-RevId: 485623855

Source-Link: https://togithub.com/googleapis/googleapis/commit/807125e7953e32535710924fe4bf0362ac93754d

Source-Link: https://togithub.com/googleapis/googleapis-gen/commit/c4ddd6d651da69658a4762c5eafff5cd8f0e25c5
Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiYzRkZGQ2ZDY1MWRhNjk2NThhNDc2MmM1ZWFmZmY1Y2Q4ZjBlMjVjNSJ9
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: cloudresourcemanager Issues related to the Resource Manager API.
Projects
None yet
Development

No branches or pull requests

3 participants