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

feat: expose timeStorageClassUpdated property of blob's #456

Merged

Conversation

athakor
Copy link
Contributor

@athakor athakor commented Aug 6, 2020

Fixes #455

@athakor athakor requested a review from frankyn August 6, 2020 17:02
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Aug 6, 2020
@codecov
Copy link

codecov bot commented Aug 6, 2020

Codecov Report

Merging #456 into master will increase coverage by 0.10%.
The diff coverage is 92.85%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #456      +/-   ##
============================================
+ Coverage     64.59%   64.69%   +0.10%     
+ Complexity      626      620       -6     
============================================
  Files            32       32              
  Lines          5236     5249      +13     
  Branches        507      510       +3     
============================================
+ Hits           3382     3396      +14     
+ Misses         1695     1694       -1     
  Partials        159      159              
Impacted Files Coverage Δ Complexity Δ
...c/main/java/com/google/cloud/storage/BlobInfo.java 88.34% <90.00%> (+0.04%) 94.00 <1.00> (+3.00)
...e/src/main/java/com/google/cloud/storage/Blob.java 83.70% <100.00%> (+1.32%) 30.00 <0.00> (ø)
...rc/main/java/com/google/cloud/storage/Storage.java 80.83% <100.00%> (+0.04%) 0.00 <0.00> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update db358c8...1d4a75b. Read the comment docs.

@athakor athakor changed the title feat: expose timeStorageClassUpdated properties of blob's feat: expose timeStorageClassUpdated property of blob's Aug 6, 2020
@product-auto-label product-auto-label bot added the api: storage Issues related to the googleapis/java-storage API. label Aug 21, 2020
@@ -276,6 +277,8 @@ static CustomerEncryption fromPb(StorageObject.CustomerEncryption customerEncryp
/** Sets the blob's storage class. */
public abstract Builder setStorageClass(StorageClass storageClass);

abstract Builder setTimeStorageClassUpdated(Long timeStorageClassUpdated);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Follow the same pattern in custom time to raise an exception and implement this method to prevent breaking changes.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

@athakor athakor force-pushed the storage-blob-timeStorageClassUpdated branch from 6b72409 to 8381b00 Compare September 2, 2020 08:34
@athakor
Copy link
Contributor Author

athakor commented Sep 2, 2020

@frankyn PTAL

@frankyn frankyn merged commit 57853ec into googleapis:master Sep 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: storage Issues related to the googleapis/java-storage API. cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Expose timeStorageClassUpdated property of the blob's
2 participants