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

Export of stats don't work when too big results #2560

Closed
pierrehenri-dauvergne opened this issue Dec 19, 2024 · 1 comment · Fixed by #2572
Closed

Export of stats don't work when too big results #2560

pierrehenri-dauvergne opened this issue Dec 19, 2024 · 1 comment · Fixed by #2572

Comments

@pierrehenri-dauvergne
Copy link
Collaborator

Ofsep found another issue when requesting an export of statistics with lots of data, for instance with field :
Study to include : Cohorte Principale.|CISCO.|Cohorte VHD.*

Tested it on ofsep-prod, had the same result.
Tested with one specific study to include and it worked well.
Tested directly on ofsep-prod database and I received more than a million lines, >1Go in a file.

Is there a limit of the maximum size that we can request or write in a file for download (or maximum size of folder where the stats are stored) ? Do you know something about this @michaelkain ?

@pierrehenri-dauvergne pierrehenri-dauvergne changed the title Export of stats don't work when too many fields Export of stats don't work when too big results Dec 19, 2024
@michaelkain
Copy link
Contributor

Hi @pierrehenri-dauvergne, I think it is important to fix this issue and it would be great if we could do so.
I wonder, why we have not tested it with ofsep data, as only they use this feature.
MS Dataset microservice triggers a stored procedure on the database via query.execute and the result list
is written into a buffer and stored as file on the disk. So, what is the exception, where is the problem? during the creation of the zip file? during the download? is it a memory issue? Looking forward to your solution and with kind regards, Michael

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

Successfully merging a pull request may close this issue.

2 participants