Skip to content
This repository was archived by the owner on Sep 25, 2026. It is now read-only.
This repository was archived by the owner on Sep 25, 2026. It is now read-only.

Merge Storage-NIO library into this repository. #242

Description

@frankyn

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.

Activity

  1. self-assigned this
    on Apr 13, 2020
  2. changed the title [-]Merge Storage-NIO library into Storage repository.[/-] [+]Merge Storage-NIO library into this repository.[/+] on Apr 13, 2020
  3. added
    type: feature request‘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.
    and removed
    triage meI really want to be triaged.
    on Apr 20, 2020
  4. frankyn commented on Apr 20, 2020

    @frankyn
    ContributorAuthor

    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.

  5. added
    status: investigatingThe issue is under investigation, which is determined to be non-trivial.
    and removed
    🚨This issue needs some love.
    on Apr 20, 2020
  6. dmitry-fa commented on May 20, 2020

    @dmitry-fa
    Contributor

    @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.

  7. dmitry-fa commented on Jun 2, 2020

    @dmitry-fa
    Contributor

    Suggested fix #345 creates a copy of FakeStorageRpc from 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 FakeStorageRpc class without adding extra dependencies.

  8. self-assigned this
    on Jun 8, 2020
  9. dmitry-fa commented on Jun 8, 2020

    @dmitry-fa
    Contributor

    FakeStorageRpc from #345 is very NIO specific, unlikely it will be used by anyone else. #351 provides a more generic approach: StorageRpcTestBase class implementing StorageRpc. FakeStorageRpc can extend StorageRpcTestBase.

    cc: @frankyn

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

api: storageIssues related to the googleapis/java-storage API.priority: p1Important issue which blocks shipping the next release. Will be fixed prior to next release.status: investigatingThe issue is under investigation, which is determined to be non-trivial.type: feature request‘Nice-to-have’ improvement, new feature or different behavior or design.

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions