wphyojpl opened a new pull request #43: URL: https://github.com/apache/incubator-sdap-ingester/pull/43
- add class to store tile blob in S3 in addition to Cassandra - update configuration to accept either S3 or Cassandra - refactor main method to avoid code duplication - TODO: add test case - TODO: AWS credentials ### How SDAP handles S3 credentials - - Query repo already has S3 class to extract data from S3: https://github.com/apache/incubator-sdap-nexus/blob/master/data-access/nexustiles/dao/S3Proxy.py - Collection Manager also has a class to observe files in S3 bucket to be ingested: https://github.com/apache/incubator-sdap-ingester/blob/dev/collection_manager/collection_manager/services/S3Observer.py - Both classes assume AWS credentials are handled at a different place since they don't have code to do so. - How are they being handled as we will need that info to write a test case. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@sdap.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
