Skip to content
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

import process crashes with invalid file (no errors reported) #74

Open
matthewhegarty opened this issue Nov 26, 2024 · 3 comments · May be fixed by #71
Open

import process crashes with invalid file (no errors reported) #74

matthewhegarty opened this issue Nov 26, 2024 · 3 comments · May be fixed by #71

Comments

@matthewhegarty
Copy link

matthewhegarty commented Nov 26, 2024

I am running the example application on Docker. I find that the process crashes when using an incomplete 'artists' file. I don't see any error message or warnings, the celery container eventually crashes with 'out of memory' error.

To reproduce (I have a branch here which runs the application in Docker - docker compose up --build):

  1. Go to Artist import
  2. Leave Resource on default of 'ArtistResourceWithM2M'
  3. Choose the attached file (change name to 'artists.csv')
  4. Click import

You will see the following requests:

image

In my Docker instance (running on MacOS) I see this error after about 5 mins:

[ERROR] Worker (pid:19) was sent SIGKILL! Perhaps out of memory?

No other errors are reported.

If you import the file 'artists-sm.csv' it runs fine (ensure an 'Instrument' FK exists with id=1).

artists.csv.txt
artists-sm.csv.txt

@matthewhegarty
Copy link
Author

btw - it's taken me a little while to get the test_project up and running - I'm happy to add this to the documentation if that's of interest?

@Eg0ra Eg0ra linked a pull request Nov 27, 2024 that will close this issue
@Eg0ra
Copy link
Contributor

Eg0ra commented Nov 27, 2024

Hi @matthewhegarty ,

I found out the reason for out of memory, it happens when trying to render template. I removed extra loop, please try, will this fix your error?

@Eg0ra Eg0ra linked a pull request Nov 27, 2024 that will close this issue
@matthewhegarty
Copy link
Author

Thanks - AFAICT it has the same issue. I merged your branch and pushed here.

btw - am in the django-import-export discord if it helps.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants