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

DICOM upload via the UI fails if more than 20 files are selected #13

Open
Christian-e-S opened this issue Mar 24, 2021 · 6 comments
Open
Assignees

Comments

@Christian-e-S
Copy link

Dear All,

The DICOM upload function via the UI works very well if I select only a small number of files (e.g., 10 files).

When I select more than 20 files, the status bars of all the uploads get stuck, and the server logs show a "java.nio.channels.ClosedChannelException" for each of the uploads.

image

Could it be, that a there are too many concurrent data base operations?

I have included the stack trace

stack_trace_ClosedChannelException.txt

Used Image:
dcm4che/dcm4chee-arc-psql:5.23.1-secure

@Christian-e-S
Copy link
Author

Christian-e-S commented Mar 24, 2021

I have verified that this issue exists also in dcm4che/dcm4chee-arc-psql:5.23.2-secure.

Would it be a solution to restrict the UI that it calls the POST request only for a limited set of images at a time? What I mean is when the user selects 300 images to upload, the UI would first upload the first set of 10 images, after completion the next set of 10, ... and so on. That would minimize the number of required open connections.

What do you think?

@gunterze
Copy link
Contributor

You may try to increase the number of available threads by ENV WILDFLY_EXECUTER_MAX_THREADS.

There is an open issue to Upload multiple DICOM/Bulkdata Objects in one STOW-RS request which would significantly reduce the server load.

@Christian-e-S
Copy link
Author

Increasing ENV WILDFLY_EXECUTER_MAX_THREADS did not resolve the issue, but Upload multiple DICOM/Bulkdata Objects in one STOW-RS request sounds like an excellent approach.

Thanks!

@gunterze
Copy link
Contributor

gunterze commented Mar 24, 2021

Optimally, the UI shall support both: packing multiple objects in one request and limiting the maximal number of concurrent requests to a configurable number.

Will also investigate, what actually causes the failure on server side....

@shral
Copy link

shral commented Dec 2, 2021

I just tested on my side, I could upload 47 files without problem, but anyway I could work on limiting the maximal active services at once. Sending all DICOM files in one request will remove the feature of showing the load process for each file the question is if we really want that.
image

@jason1122g
Copy link

jason1122g commented Dec 20, 2021

Same issue on my machine, but the issue only appears if files are more than 16
This issue might be irrelevant with file size, I have tried files <= 16 with larger total size and the issue does not appear
I have also tried to increase WILDFLY_EXECUTER_MAX_THREADS but not working 🤔

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

No branches or pull requests

4 participants