Skip to content

feat: add delimiter BlobListOption - #102

Merged
frankyn merged 4 commits into
googleapis:masterfrom
athakor:storage-99
Feb 14, 2020
Merged

feat: add delimiter BlobListOption#102
frankyn merged 4 commits into
googleapis:masterfrom
athakor:storage-99

Conversation

@athakor

Copy link
Copy Markdown
Contributor

Fixes#99

@googlebotgooglebot added the cla: yes This human has signed the Contributor License Agreement. label Jan 30, 2020
@codecov

codecovBot commented Jan 30, 2020

Copy link
Copy Markdown

Codecov Report

Merging #102 into master will increase coverage by 0.32%.
The diff coverage is n/a.

Impacted file tree graph

@@ Coverage Diff @@## master #102 +/- ##
============================================
+ Coverage 63.30% 63.62% +0.32% - Complexity 528 539 +11 
============================================
Files 30 30 Lines 4720 4729 +9 Branches 450 426 -24 ============================================
+ Hits 2988 3009 +21 + Misses 1571 1562 -9 + Partials 161 158 -3 
Impacted FilesCoverage ΔComplexity Δ
...in/java/com/google/cloud/storage/StorageBatch.java88.00% <0.00%> (-4.00%)13.00% <0.00%> (ø%)
...gle/cloud/storage/testing/RemoteStorageHelper.java64.46% <0.00%> (-0.58%)9.00% <0.00%> (ø%)
...main/java/com/google/cloud/storage/BucketInfo.java80.85% <0.00%> (-0.20%)84.00% <0.00%> (ø%)
...e/src/main/java/com/google/cloud/storage/Blob.java82.24% <0.00%> (-0.11%)29.00% <0.00%> (-1.00%)
...om/google/cloud/storage/spi/v1/HttpStorageRpc.java1.64% <0.00%> (+<0.01%)1.00% <0.00%> (ø%)⬆️
...rc/main/java/com/google/cloud/storage/Storage.java80.68% <0.00%> (+0.63%)0.00% <0.00%> (ø%)⬆️
...n/java/com/google/cloud/storage/SignatureInfo.java81.50% <0.00%> (+0.81%)19.00% <0.00%> (+2.00%)⬇️
...ain/java/com/google/cloud/storage/StorageImpl.java78.67% <0.00%> (+1.85%)120.00% <0.00%> (+10.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 195096c...62a567b. Read the comment docs.

@athakorathakor added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 4, 2020
@yoshi-kokoroyoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 4, 2020
@athakor
athakor requested a review from frankynFebruary 7, 2020 08:45
}

/** Returns an option to set a delimiter '/'. */
public static BlobListOption delimiter() {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Delimiter can be user defined, generally '/' is the one used most often, but I'd like to keep this open to other delimiters.

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

done

@athakorathakor added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 10, 2020
@yoshi-kokoroyoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 10, 2020
@athakor

Copy link
Copy Markdown
ContributorAuthor

@frankyn PTAL

@frankynfrankyn left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@frankyn
frankyn merged commit b30a675 into googleapis:masterFeb 14, 2020
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yesThis human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add delimiter BlobListOption

4 participants

@athakor@frankyn@googlebot@yoshi-kokoro