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.

Sign up for GitHub

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

Jump to bottom

Storage should be stopped immediately if a storage is already going on with the same name #1013

Closed
bruntib opened this issue Oct 9, 2017 · 0 comments
Closed

Storage should be stopped immediately if a storage is already going on with the same name #1013

bruntib opened this issue Oct 9, 2017 · 0 comments
Assignees
@bruntib
Labels
enhancement 🌟 server 🖥️
Milestone
6.1.1

Comments

@bruntib
Copy link
Contributor

bruntib commented Oct 9, 2017

The mass storage has two phases: storing the files and storing the results. The latter one happens in a separate transaction. At the beginning of this transaction it is checked whether there is another transaction which stores the same run name. We get an error message for this but only after the storage of files. If a large .zip is uploaded then it takes some time. We should do the checking before the file storage.
@gyorb gyorb modified the milestones: release 6.0.2, release 6.1, release 6.2, 6.1.1 Oct 9, 2017
@bruntib bruntib self-assigned this Oct 17, 2017
bruntib added a commit to bruntib/codechecker that referenced this issue Oct 17, 2017
@bruntib
Check at the beginning of store if another is ongoing
784096e 
We shouldn't wait for file storage to notice that another storage with
the same name is already going.
Fixes Ericsson#1013
@bruntib bruntib mentioned this issue Oct 17, 2017
Merged
bruntib added a commit to bruntib/codechecker that referenced this issue Oct 17, 2017
@bruntib
Check at the beginning of store if another is ongoing
71c465d 
We shouldn't wait for file storage to notice that another storage with
the same name is already going.
Fixes Ericsson#1013
bruntib added a commit to bruntib/codechecker that referenced this issue Oct 17, 2017
@bruntib
Check at the beginning of store if another is ongoing
3fca315 
We shouldn't wait for file storage to notice that another storage with
the same name is already going.
Fixes Ericsson#1013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bruntib bruntib

Labels
enhancement 🌟 server 🖥️
Projects
None yet
Milestone
6.1.1
Development

No branches or pull requests
2 participants
@gyorb @bruntib