lendup / fs2-blobstore

Minimal, idiomatic, stream-based Scala interface for key/value store implementations
Apache License 2.0
92 stars 26 forks source link

migrating s3 put to tm upload #28

Closed loran-steinberger closed 5 years ago

loran-steinberger commented 5 years ago

TODO: bump the version.

Relevant:

Q: How much data can I store in Amazon S3?

The total volume of data and number of objects you can store are unlimited. Individual Amazon S3 objects can range in size from a minimum of 0 bytes to a maximum of 5 terabytes. The largest object that can be uploaded in a single PUT is 5 gigabytes. For objects larger than 100 megabytes, customers should consider using the Multipart Upload capability.

AWS guidance is use TransferManager/Mulitpart upload for anything over 100mb

matthewgraf commented 5 years ago

I believe we have tests covering this. There are some integration tests that are disabled but can be run locally (some of them are designed to actually interact with s3, Box, or whatever store you are working with) Also I'm not sure what is standard practice with versioning, but I would think we probably want to bump the version number for any change

srki-lendup commented 5 years ago

Agree with Matt on version bump.

stew commented 5 years ago

if you are changing a public signature it should be a version bump

rolandomanrique commented 5 years ago

I believe we have tests covering this. There are some integration tests that are disabled but can be run locally (some of them are designed to actually interact with s3, Box, or whatever store you are working with)

@matthewgraf I set up tests to run against Minio (dockerized S3) so S3 tests are running in travis (I filed issue #25 to fix codecov publishing).

Agree we should bump version.

Otherwise this looks good to me.

matthewgraf commented 5 years ago

Agree we should bump version.

@rolandomanrique where is the version changed? I was looking over this with @lumengxi and it looks like the version in version.sbt is out of sync with the latest on Sonatype as well as the latest in the repo "Releases" tab

loran-steinberger commented 5 years ago

Reconciling all version references to the same- 0.3.0

loran-steinberger commented 5 years ago

release @rolandomanrique