Athens: Proposal: get rid of the storage.Connector

Created on 25 Jul 2018  路  4Comments  路  Source: gomods/athens

I'm not sure about this one. Do we need a separate Connect() function, or is it ok to connect automatically when we create a new Saver, Lister, etc...?

good first issue proposal storage

Most helpful comment

I'll work on this one.

All 4 comments

I like just connecting on creation, this gives the caller a ready to use Saver etc

if we have connect, we should have disconnect as well. it's not right to give a fork and keep the knife.
but i think we should go with auto-connect up to the point we need more control over this.

I'll work on this one.

Closing #457 is in

Was this page helpful?
0 / 5 - 0 ratings