Repository navigation
Merge Storage-NIO library into this repository. #242
Description
Activity
- addedapi: storageIssues related to the googleapis/java-storage API.Issues related to the googleapis/java-storage API.
on Apr 13, 2020 - changed the title
[-]Merge Storage-NIO library into Storage repository.[/-][+]Merge Storage-NIO library into this repository.[/+]on Apr 13, 2020 - addedtriage meI really want to be triaged.I really want to be triaged.🚨This issue needs some love.This issue needs some love.
on Apr 14, 2020 - addedtype: feature request‘Nice-to-have’ improvement, new feature or different behavior or design.‘Nice-to-have’ improvement, new feature or different behavior or design.priority: p1Important issue which blocks shipping the next release. Will be fixed prior to next release.Important issue which blocks shipping the next release. Will be fixed prior to next release.and removedtriage meI really want to be triaged.I really want to be triaged.
on Apr 20, 2020 After discussing with @chingor13, the more likely approach is to separate out testing related code used by the Storage-Nio library into a separate artifact called google-cloud-storage-testing instead of merging Storage-NIO library into this repository.
Reacted by Ajit Thakor- addedstatus: investigatingThe issue is under investigation, which is determined to be non-trivial.The issue is under investigation, which is determined to be non-trivial.and removed🚨This issue needs some love.This issue needs some love.
on Apr 20, 2020 @frankyn, I this is about exporting FakeStorageRpc for the sake of testing I can take care of this issue to move it on. googleapis/google-cloud-java#12653 and #179 are currently blocked by this issue.
Suggested fix #345 creates a copy of
FakeStorageRpcfrom Storage-Nio in Storage. Because the main storage artifact already includes a testing package (com.google.cloud.storage.testing) the new class is placed there. No extra artifact is required.I verified, Storage-Nio could switch to the new
FakeStorageRpcclass without adding extra dependencies.
Currently have two repositories that must be merged into one library.
https://github.com/googleapis/java-storage-nio needs to be merged into this repository to unblock changes to the underlying internal only StorageRpc interface. Otherwise we can accidentally break users.