Remove default contentType from Storage.compose and Bucket.create #762

mziccard · 2016-03-18T14:12:09Z

This fixes #288

coveralls · 2016-03-18T14:19:01Z

Coverage increased (+0.03%) to 83.228% when pulling 8840b33 on mziccard:remove-compose-content-type into 7d48ec7 on GoogleCloudPlatform:master.

gcloud-java-storage/src/main/java/com/google/gcloud/storage/Bucket.java

+   * @throws StorageException upon failure
+   */
+  public Blob create(String blob, byte[] content, BlobTargetOption... options) {
+    BlobInfo blobInfo = BlobInfo.builder(BlobId.of(name(), blob)).build();


aozarov · 2016-03-18T15:29:46Z

Looks great. Thanks for doing that. Should we also add something to the integration test?
Also, I suspect this enforcment is no longer needed?

- update CopyRequest to hold both targetId and targetInfo - avoid sending storage object in StorageRpc.rewrite when only targetId is set - refactor CopyWriter and state - add integration tests

coveralls · 2016-03-19T09:17:31Z

Coverage increased (+1.1%) to 84.321% when pulling 22636e2 on mziccard:remove-compose-content-type into 7d48ec7 on GoogleCloudPlatform:master.

gcloud-java-storage/src/main/java/com/google/gcloud/storage/Storage.java

@@ -973,7 +972,8 @@ public static Builder builder() {
      private final Set<BlobSourceOption> sourceOptions = new LinkedHashSet<>();
      private final Set<BlobTargetOption> targetOptions = new LinkedHashSet<>();
      private BlobId source;
-      private BlobInfo target;
+      private BlobId targetId;


aozarov · 2016-03-19T18:46:12Z

Nice, few more comments/suggestions.

coveralls · 2016-03-20T20:16:01Z

Coverage increased (+1.1%) to 84.318% when pulling 1855ae1 on mziccard:remove-compose-content-type into 7d48ec7 on GoogleCloudPlatform:master.

gcloud-java-storage/src/main/java/com/google/gcloud/storage/CopyWriter.java

@@ -171,9 +175,9 @@ public RewriteResponse call() {
      private final StorageOptions serviceOptions;
      private final BlobId source;
      private final Map<StorageRpc.Option, ?> sourceOptions;
-      private final BlobId targetId;
+      private final boolean overrideInfo;
+      private BlobInfo target;


aozarov · 2016-03-20T21:05:30Z

LGTM. Feel free to merge after considering the 2 comments above.

…iter's StateImpl.target

coveralls · 2016-03-20T21:48:59Z

Coverage increased (+1.1%) to 84.318% when pulling 5dd0292 on mziccard:remove-compose-content-type into 7d48ec7 on GoogleCloudPlatform:master.

Remove default contentType from Storage.compose and Bucket.create

- [ ] Regenerate this pull request now. PiperOrigin-RevId: 454027580 Source-Link: googleapis/googleapis@1b22277 Source-Link: https://github.com/googleapis/googleapis-gen/commit/e04cea20d0d12eb5c3bdb360a9e72b654edcb638 Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiZTA0Y2VhMjBkMGQxMmViNWMzYmRiMzYwYTllNzJiNjU0ZWRjYjYzOCJ9 chore(deps): upgrade gapic-generator-java to 2.8.0 and update gax-java to 2.18.1 PiperOrigin-RevId: 450543911 Source-Link: googleapis/googleapis@5528344 Source-Link: https://github.com/googleapis/googleapis-gen/commit/9f6775cab1958982b88967a43e5e806af0f135db Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiOWY2Nzc1Y2FiMTk1ODk4MmI4ODk2N2E0M2U1ZTgwNmFmMGYxMzVkYiJ9

🤖 I have created a release *beep* *boop* --- ## [1.3.0](googleapis/java-resourcemanager@v1.2.11...v1.3.0) (2022-04-18) ### Features * AuditConfig for IAM v1 ([googleapis#747](googleapis/java-resourcemanager#747)) ([6806574](googleapis/java-resourcemanager@6806574)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).

Making CLIRR not required. The version bumps are now controlled by the Release Please and OwlBot. The CL authors create appropriate change description to control major version bumps.

🤖 I have created a release *beep* *boop* --- ## [0.123.2](googleapis/java-datalabeling@v0.123.1...v0.123.2) (2022-07-13) ### Bug Fixes * enable longpaths support for windows test ([#1485](https://github.com/googleapis/java-datalabeling/issues/1485)) ([#761](googleapis/java-datalabeling#761)) ([a57a787](googleapis/java-datalabeling@a57a787)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).

Fixes #n/a☕️ Changed the network name being created We were seeing the error where we ran into too many clusters being created with the `default` vpc network, which was also being by other java CI flows not in this repo.

…1.1 (#762) [![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:libraries-bom](https://cloud.google.com/java/docs/bom) ([source](https://github.com/googleapis/java-cloud-bom)) | `26.1.0` -> `26.1.1` | [![age](https://badges.renovateapi.com/packages/maven/com.google.cloud:libraries-bom/26.1.1/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/maven/com.google.cloud:libraries-bom/26.1.1/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/maven/com.google.cloud:libraries-bom/26.1.1/compatibility-slim/26.1.0)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/maven/com.google.cloud:libraries-bom/26.1.1/confidence-slim/26.1.0)](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**: Renovate will not automatically rebase this PR, because other commits have been found. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, click this checkbox. ⚠ **Warning**: custom changes will be lost. --- 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-iot).

…1.4 (#762) [![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:libraries-bom](https://cloud.google.com/java/docs/bom) ([source](https://github.com/googleapis/java-cloud-bom)) | `26.1.3` -> `26.1.4` | [![age](https://badges.renovateapi.com/packages/maven/com.google.cloud:libraries-bom/26.1.4/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/maven/com.google.cloud:libraries-bom/26.1.4/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/maven/com.google.cloud:libraries-bom/26.1.4/compatibility-slim/26.1.3)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/maven/com.google.cloud:libraries-bom/26.1.4/confidence-slim/26.1.3)](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**: Renovate will not automatically rebase this PR, because other commits have been found. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, check this box --- 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-servicedirectory).

🤖 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).

mziccard added 2 commits March 18, 2016 14:25

Remove default contentType from Storage's compose

0a113cd

Remove default contentType from Bucket's create

8840b33

mziccard added type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns. api: storage Issues related to the Cloud Storage API. labels Mar 18, 2016

googlebot added the cla: yes This human has signed the Contributor License Agreement. label Mar 18, 2016

aozarov reviewed Mar 18, 2016
View reviewed changes

Remove checks for contentType in CopyRequest

22636e2

- update CopyRequest to hold both targetId and targetInfo - avoid sending storage object in StorageRpc.rewrite when only targetId is set - refactor CopyWriter and state - add integration tests

aozarov reviewed Mar 19, 2016
View reviewed changes

mziccard added 2 commits March 20, 2016 21:05

Refactor CopyRequest: remove targetId, add boolean overrideInfo

3e5db7a

Add better javadoc to Storage.copy and CopyWriter

1855ae1

aozarov reviewed Mar 20, 2016
View reviewed changes

Rephrase RewriteChannel and Storage.copy javadoc. Add final to CopyWr…

5dd0292

…iter's StateImpl.target

mziccard added a commit that referenced this pull request Mar 20, 2016

Merge pull request #762 from mziccard/remove-compose-content-type

fa64fa5

Remove default contentType from Storage.compose and Bucket.create

mziccard merged commit fa64fa5 into googleapis:master Mar 20, 2016

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Remove default contentType from Storage.compose and Bucket.create #762

Remove default contentType from Storage.compose and Bucket.create #762

mziccard commented Mar 18, 2016

coveralls commented Mar 18, 2016

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

aozarov commented Mar 18, 2016

coveralls commented Mar 19, 2016

This comment was marked as spam.

aozarov commented Mar 19, 2016

coveralls commented Mar 20, 2016

This comment was marked as spam.

This comment was marked as spam.

aozarov commented Mar 20, 2016

coveralls commented Mar 20, 2016

Remove default contentType from Storage.compose and Bucket.create #762

Remove default contentType from Storage.compose and Bucket.create #762

Conversation

mziccard commented Mar 18, 2016

coveralls commented Mar 18, 2016

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

aozarov commented Mar 18, 2016

coveralls commented Mar 19, 2016

This comment was marked as spam.

aozarov commented Mar 19, 2016

coveralls commented Mar 20, 2016

This comment was marked as spam.

This comment was marked as spam.

aozarov commented Mar 20, 2016

coveralls commented Mar 20, 2016