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

server: Wrap ExternalStorage factory methods in externalStorageBuilder struct #50293

Merged
merged 1 commit into from
Jun 18, 2020

Commits on Jun 17, 2020

  1. server: Wrap ExternalStorage factory methods in externalStorageBuilde…

    …r struct
    
    Previously, we initialized the ExternalStorage factory methods on
    creation of a NewServer() as all the required config params were
    ready-to-use.
    
    With future work related to user scoped storage requiring access to the
    underlying storage.Engine, this change introduces a wrapper around these
    factory methods. Using a builder struct allows us to split the
    "creation" and "initialization" of the builder between the NewServer()
    and Start() methods respectively. This allows for params which are only
    initialized on server.Start() to be propogated to the builder for future
    use.
    
    This is part of a gradual refactor of the ExternalStorage factory
    interface and is primarily to unblock development of cockroachdb#47211.
    
    Release note: None
    adityamaru committed Jun 17, 2020
    Configuration menu
    Copy the full SHA
    c3b8bcd View commit details
    Browse the repository at this point in the history