[GitHub] incubator-predictionio issue #371: [PIO-61] Add S3 Model Data Repository

2017-06-04 Thread marevol
Github user marevol commented on the issue: https://github.com/apache/incubator-predictionio/pull/371 I would like to merge S3 support this week. For travis testing, it passed in https://travis-ci.org/jpioug/incubator-predictionio/builds/235569236 though apache-travis was

[GitHub] incubator-predictionio issue #371: [PIO-61] Add S3 Model Data Repository

2017-05-24 Thread marevol
Github user marevol commented on the issue: https://github.com/apache/incubator-predictionio/pull/371 Added LocalStack for S3 testing. LocalStack/moto seems to have a problem about ETag handling... I added DISABLE_CHUNKED_ENCODING option to avoid it in S3 testing. --- If

[GitHub] incubator-predictionio issue #371: [PIO-61] Add S3 Model Data Repository

2017-05-03 Thread dszeto
Github user dszeto commented on the issue: https://github.com/apache/incubator-predictionio/pull/371 @marevol Would you like to add some integration test for this? https://github.com/jubos/fake-s3 or https://github.com/atlassian/localstack might be useful for mocking S3 locally