PavlidisLab / Gemma

Genomics data re-analysis
Apache License 2.0
23 stars 6 forks source link

Splitting an experiment should be annotated with @Transactional #194

Open arteymix opened 3 years ago

arteymix commented 3 years ago

This is regarding SplitExperimentService.split.

I tried to make it @Transactional at some point, but it didn't work properly. Multiple things can go wrong in there since it's a complex operation that can leave the original experiment or some of its parts in a bad state.

I noticed this while reviewing SonarQube report.

arteymix commented 1 month ago

Splitting yes, but not postprocessing afterwards.