scitran / core

RESTful API
https://scitran.github.io
MIT License
18 stars 18 forks source link

Remove unnecessary analysis input file update #1067

Closed nagem closed 6 years ago

nagem commented 6 years ago

If a job fails and is respawned, an analysis will attempt to update the job id to the most recently spawned job. If that update is required, old code would also update the analysis's input files to match the new job's inputs. Code prevents a change in inputs from one job attempt to the next, so this is unnecessary. It was causing issues in situations where inputs did not exist for a job, so it easier to just remove it.

Review Checklist

codecov-io commented 6 years ago

Codecov Report

Merging #1067 into master will decrease coverage by 0.01%. The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1067      +/-   ##
==========================================
- Coverage   91.19%   91.18%   -0.02%     
==========================================
  Files          50       50              
  Lines        6931     6922       -9     
==========================================
- Hits         6321     6312       -9     
  Misses        610      610