-
Notifications
You must be signed in to change notification settings - Fork 52
backend doesn't support absolute paths: Support for non local filesystem #31
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
Comments
Because of the way this library leans on |
I have integrated Django Data Wizard with S3 before, but that was with very large (multi-GB) CSV files and some heavy customization to parallelize the import across several Lambda workers. For the simple case with # myapp/wizard.py
from itertable import ExcelFileIter # or whichever format
from data_wizard.loaders import FileLoader
class ExcelS3Iter(ExcelFileIter):
def load(self):
response = s3.get_object(
Bucket=self.bucket,
Key=self.filename,
)
self.file = response['Body'] # I think this is already file-like
class S3Loader(FileLoader):
def load_iter(self):
return ExcelS3Iter(
bucket=settings.AWS_STORAGE_BUCKET_NAME,
filename=self.file.name,
)
# myproject/settings.py
DATA_WIZARD = {
'LOADER': 'myapp.wizard.S3Loader',
} I believe no other customization of The goal with having |
Actually, I think a better approach in this specific case is to update I haven't tested this with |
This fix has been released. |
When using data-wizard with S3 as default storage I get the following error.
Django 1.11
Python 2.7
The text was updated successfully, but these errors were encountered: