POSC support for Storage#1670
Conversation
|
Changes Unknown when pulling 37f08e9 on shinfan:master into ** on GoogleCloudPlatform:master**. |
|
|
||
| abstract Builder setCustomerEncryption(CustomerEncryption customerEncryption); | ||
|
|
||
|
|
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
| private static final String KEY_SHA256 = "keySha"; | ||
| private static final CustomerEncryption CUSTOMER_ENCRYPTION = | ||
| new CustomerEncryption(ENCRYPTION_ALGORITHM, KEY_SHA256); | ||
| private static final String STORAGE_CLASS = "COLDLINE"; |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
7840605 to
37f08e9
Compare
|
@michaelbausor PTAL |
michaelbausor
left a comment
There was a problem hiding this comment.
One comment, otherwise LGTM
| /** | ||
| * String constants for the names of the storage classes. | ||
| */ | ||
| public class StorageClass { |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
|
@garrettjonesgoogle PTAL |
| * Enums for the storage classes. | ||
| * See https://cloud.google.com/storage/docs/storage-classes for details. | ||
| */ | ||
| public enum StorageClass { |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
| Bucket bucket = storage.create(BucketInfo.newBuilder(bucketName) | ||
| // See here for possible values: http://g.co/cloud/storage/docs/storage-classes | ||
| .setStorageClass("COLDLINE") | ||
| .setStorageClass(StorageClass.COLDLINE) |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
| * <a href="https://cloud.google.com/storage/docs/storage-classes">here</a>. | ||
| */ | ||
| public abstract Builder setStorageClass(String storageClass); | ||
| public abstract Builder setStorageClass(StorageClass storageClass); |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
| */ | ||
| @Deprecated | ||
| public abstract Builder storageClass(String storageClass); | ||
| public abstract Builder storageClass(StorageClass storageClass); |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
| @@ -0,0 +1,48 @@ | |||
| /* | |||
| * Copyright 2015 Google Inc. All Rights Reserved. | |||
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
|
|
||
| @Override | ||
| public Builder setStorageClass(String storageClass) { | ||
| public Builder setStorageClass(StorageClass storageClass) { |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
|
@garrettjonesgoogle PTAL |
|
LGTM |
|
Changes Unknown when pulling 59499dc on shinfan:master into ** on GoogleCloudPlatform:master**. |
…#1670) * chore: Update generation configuration at Thu Aug 1 02:18:43 UTC 2024 * chore: Update generation configuration at Fri Aug 2 02:14:46 UTC 2024
…ent-bom to v1.46.1 (#1670)
…googleapis#1670) * chore: Update generation configuration at Thu Aug 1 02:18:43 UTC 2024 * chore: Update generation configuration at Fri Aug 2 02:14:46 UTC 2024
…ent-bom to v1.46.1 (#1670)
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
…o v1.123.19 (#1670) [](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [com.google.cloud:google-cloud-pubsub](https://togithub.com/googleapis/java-pubsub) | `1.123.18` -> `1.123.19` | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | --- ### Release Notes <details> <summary>googleapis/java-pubsub (com.google.cloud:google-cloud-pubsub)</summary> ### [`v1.123.19`](https://togithub.com/googleapis/java-pubsub/blob/HEAD/CHANGELOG.md#112319-2023-07-25) [Compare Source](https://togithub.com/googleapis/java-pubsub/compare/v1.123.18...v1.123.19) ##### Dependencies - Update dependency com.google.cloud:google-cloud-bigquery to v2.30.1 ([#​1656](https://togithub.com/googleapis/java-pubsub/issues/1656)) ([4fe4bc4](https://togithub.com/googleapis/java-pubsub/commit/4fe4bc49d82753e3e185f0a2c4694d80e48808f7)) - Update dependency com.google.cloud:google-cloud-core to v2.21.1 ([#​1662](https://togithub.com/googleapis/java-pubsub/issues/1662)) ([e10292c](https://togithub.com/googleapis/java-pubsub/commit/e10292c16f24870a91584f716545ff210302a7bc)) - Update dependency com.google.cloud:google-cloud-shared-dependencies to v3.13.1 ([#​1663](https://togithub.com/googleapis/java-pubsub/issues/1663)) ([50f4ff1](https://togithub.com/googleapis/java-pubsub/commit/50f4ff10140699a02d1ec550b05ff6d1aa39c762)) - Update dependency org.junit.vintage:junit-vintage-engine to v5.10.0 ([#​1666](https://togithub.com/googleapis/java-pubsub/issues/1666)) ([d8712a3](https://togithub.com/googleapis/java-pubsub/commit/d8712a31459b8f9497dff7a866c8a8795ca60af9)) - Update dependency org.xerial.snappy:snappy-java to v1.1.10.3 ([#​1665](https://togithub.com/googleapis/java-pubsub/issues/1665)) ([ad1c515](https://togithub.com/googleapis/java-pubsub/commit/ad1c5157eadc4bf2d43b7dc332ac4150d732912d)) </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, 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://developer.mend.io/github/googleapis/java-pubsub). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNi4xMS4wIiwidXBkYXRlZEluVmVyIjoiMzYuMTEuMCIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->
…o v1.123.19 (#1670) [](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [com.google.cloud:google-cloud-pubsub](https://togithub.com/googleapis/java-pubsub) | `1.123.18` -> `1.123.19` | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | --- ### Release Notes <details> <summary>googleapis/java-pubsub (com.google.cloud:google-cloud-pubsub)</summary> ### [`v1.123.19`](https://togithub.com/googleapis/java-pubsub/blob/HEAD/CHANGELOG.md#112319-2023-07-25) [Compare Source](https://togithub.com/googleapis/java-pubsub/compare/v1.123.18...v1.123.19) ##### Dependencies - Update dependency com.google.cloud:google-cloud-bigquery to v2.30.1 ([#​1656](https://togithub.com/googleapis/java-pubsub/issues/1656)) ([15d7ffe](https://togithub.com/googleapis/java-pubsub/commit/15d7ffe46df78ec400839e601864bc955c5ea20e)) - Update dependency com.google.cloud:google-cloud-core to v2.21.1 ([#​1662](https://togithub.com/googleapis/java-pubsub/issues/1662)) ([5d06218](https://togithub.com/googleapis/java-pubsub/commit/5d062187ed1fd463d122491a4b4747383c67faca)) - Update dependency com.google.cloud:google-cloud-shared-dependencies to v3.13.1 ([#​1663](https://togithub.com/googleapis/java-pubsub/issues/1663)) ([f677004](https://togithub.com/googleapis/java-pubsub/commit/f67700468e3975665f8544c90663714f73dd3471)) - Update dependency org.junit.vintage:junit-vintage-engine to v5.10.0 ([#​1666](https://togithub.com/googleapis/java-pubsub/issues/1666)) ([47f8a34](https://togithub.com/googleapis/java-pubsub/commit/47f8a3465a81228aebe02486c5edffa3f89ac59f)) - Update dependency org.xerial.snappy:snappy-java to v1.1.10.3 ([#​1665](https://togithub.com/googleapis/java-pubsub/issues/1665)) ([c557254](https://togithub.com/googleapis/java-pubsub/commit/c55725449952461d975ef36e3bf0dd7f51bab64f)) </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, 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://developer.mend.io/github/googleapis/java-pubsub). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNi4xMS4wIiwidXBkYXRlZEluVmVyIjoiMzYuMTEuMCIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->
Fixes: #1602
Tests: Unit tests, Integration tests
cc @omaray