Add ObjectId to identify stored objects #94

jboynes · 2015-06-04T16:18:26Z

For #62, add an ObjectId class as discussed. Updates to the API to use it would follow if we agree this is the way to proceed.

Conflicts: .gitignore git-demo.iml src/main/java/com/google/gcloud/RetryHelper.java

s/XXXService/XXX/g

…java into temp

enforce minium Java and Maven requirements

rename Bucket to BucketInfo and Blob to BlobInfo

rename load to readAllBytes

coveralls · 2015-06-04T16:28:17Z

Coverage increased (+0.29%) to 65.01% when pulling 3f19255 on jboynes:issue62_objectid into 0cd6f98 on GoogleCloudPlatform:master.

coveralls · 2015-06-04T16:28:17Z

Coverage increased (+0.29%) to 65.01% when pulling 3f19255 on jboynes:issue62_objectid into 0cd6f98 on GoogleCloudPlatform:master.

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

+        this.uri = uri;
+        if (uri.isAbsolute()) {
+            if (!"gs".equals(uri.getScheme())) {
+                throw new IllegalArgumentException("uri must use gs: scheme");


aozarov · 2015-06-05T00:36:05Z

This change seems to include changes from several other pull requests (#91 and #92) can we remove them from this PR?

jboynes · 2015-06-05T01:31:07Z

@aozarov Rebased this change

coveralls · 2015-06-05T01:36:56Z

Coverage increased (+0.29%) to 65.0% when pulling b93f1d3 on jboynes:issue62_objectid into 0cd6f98 on GoogleCloudPlatform:master.

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

+/**
+ * Identifier for a stored object.
+ */
+public class ObjectId {


aozarov · 2015-10-16T03:32:52Z

Closing this one in favor of #251

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

🤖 I have created a release *beep* *boop* --- ## [1.1.7](googleapis/java-optimization@v1.1.6...v1.1.7) (2022-08-06) ### Documentation * **owlbot-java:** explaining why not using formatter in pom.xml ([#1511](https://github.com/googleapis/java-optimization/issues/1511)) ([#77](googleapis/java-optimization#77)) ([7177414](googleapis/java-optimization@7177414)), closes [#1502](https://github.com/googleapis/java-optimization/issues/1502) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).

🤖 I have created a release *beep* *boop* --- ## [1.1.1](googleapis/java-dataplex@v1.1.0...v1.1.1) (2022-09-08) ### Dependencies * Update dependency com.google.cloud:google-cloud-shared-dependencies to v3.0.2 ([#94](googleapis/java-dataplex#94)) ([286ac51](googleapis/java-dataplex@286ac51)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).

* deps: update dependency typing-extensions to v4.3.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>

🤖 I have created a release *beep* *boop* --- ### Updating meta-information for bleeding-edge SNAPSHOT release. --- This PR was generated with [Release Please](https://togithub.com/googleapis/release-please). See [documentation](https://togithub.com/googleapis/release-please#release-please).

…i-cloud to v0.2.6 (#94) [![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-gke-multi-cloud](https://togithub.com/googleapis/java-gke-multi-cloud) | `0.2.4` -> `0.2.6` | [![age](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-gke-multi-cloud/0.2.6/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-gke-multi-cloud/0.2.6/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-gke-multi-cloud/0.2.6/compatibility-slim/0.2.4)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-gke-multi-cloud/0.2.6/confidence-slim/0.2.4)](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 these updates 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-gke-multi-cloud).

aozarov and others added 30 commits January 2, 2015 17:50

Automatically invoke gcd for testing

c79887b

Merge branch 'master' of https://github.com/aozarov/git-demo

12d00ef

minor change

e03e416

replace sh with bash

1ee59da

cleanup

5e5a143

more cleanup

7bc7ce1

more cleanup

66f062a

minor fix

3c80071

revert wrong formatting

8a45fb2

revert wrong formatting

eb88c65

even more cleanup

a0825e7

Merge remote-tracking branch 'origin/master'

c1bd6fa

Conflicts: .gitignore git-demo.iml src/main/java/com/google/gcloud/RetryHelper.java

revert wrong formatting

cff6d8f

Provide an option for an add with auto-id allocation.

cc001fe

add git-demo.iml

e6eda06

removing git-demo.iml

797af85

batch/transaction submit/commit returns generated keys

64a1929

Provide an option for an add with auto-id allocation.

d27b2de

replace local gcd hack with a fixed version of gcd

e68feca

replace local gcd hack with a fixed version of gcd

e262b76

Adding copyright and README and CONTRIBUTIONS files.

fa60bc9

adding contributing

acb92fc

adding travis config file

4b0bd41

Make travis run verify instead of test

7a79f5d

change maven group information

9e21d2b

adding travis credentials

c18cf24

apply changes from git-demo

beac379

return the datastore service from Batch and Transation

0ae2558

Delete .project

fdbd594

Delete .classpath

9b469f7

aozarov and others added 8 commits May 29, 2015 12:35

fix merge conflicts

19f3350

Merge pull request #79 from aozarov/temp1

930a52d

s/XXXService/XXX/g

Merge branch 'master' of git://github.com/GoogleCloudPlatform/gcloud-…

0201796

…java into temp

rename Bucket to BucketInfo and Blob to BlobInfo

f85b3d6

Merge pull request #78 from aozarov/temp

0ad1cbd

enforce minium Java and Maven requirements

Merge pull request #86 from aozarov/master

857077c

rename Bucket to BucketInfo and Blob to BlobInfo

rename load to readAllBytes

5896432

Merge pull request #89 from aozarov/temp

0cd6f98

rename load to readAllBytes

jgeewax reviewed Jun 4, 2015
View reviewed changes

Add an ObjectId class for identifying stored objects

b93f1d3

aozarov reviewed Jun 5, 2015
View reviewed changes

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

/**

* Identifier for a stored object.

*/

public class ObjectId {

This comment was marked as spam.

Sign in to view

aozarov force-pushed the master branch 2 times, most recently from be94fc7 to 462b083 Compare July 7, 2015 22:37

aozarov closed this Oct 16, 2015

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add ObjectId to identify stored objects #94

Add ObjectId to identify stored objects #94

jboynes commented Jun 4, 2015

coveralls commented Jun 4, 2015

coveralls commented Jun 4, 2015

This comment was marked as spam.

This comment was marked as spam.

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 Jun 5, 2015

jboynes commented Jun 5, 2015

coveralls commented Jun 5, 2015

This comment was marked as spam.

aozarov commented Oct 16, 2015

Add ObjectId to identify stored objects #94

Add ObjectId to identify stored objects #94

Conversation

jboynes commented Jun 4, 2015

coveralls commented Jun 4, 2015

coveralls commented Jun 4, 2015

This comment was marked as spam.

This comment was marked as spam.

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 Jun 5, 2015

jboynes commented Jun 5, 2015

coveralls commented Jun 5, 2015

This comment was marked as spam.

aozarov commented Oct 16, 2015