edx / edx-submissions

59%

master: 83%

LAST BUILD ON BRANCH efischer/soft_delete_sub

branch: efischer/soft_delete_sub

CHANGE BRANCH

x

Reset

• efischer/soft_delete_sub
• 0.0.1
• 0.0.2
• 0.0.3
• 0.0.4
• 0.0.5
• 0.0.6
• 0.0.7
• 0.1.0
• 0.1.1
• 0.1.2
• 0.1.3
• 0.1.4
• 1.0.0
• 1.1.0
• 1.1.1
• 1.1.2
• 1.1.3
• 1.1.4
• 1.1.5
• 1.1.6
• 1.2.0
• 2.0.0
• 2.0.1
• 2.0.10
• 2.0.11
• 2.0.12
• 2.0.2
• 2.0.3
• 2.0.4
• 2.0.5
• 2.0.6
• 2.0.7
• 2.0.8
• 2.0.9
• 2.1.1
• 3.0.0
• 3.0.1
• 3.0.2
• BOM-1030-django22-support
• BOM-75
• add-openedx-yaml
• aed/django1.11
• aed/fix-travis
• aed/travis-cond
• ammar/dj18-update-django-extensions
• ammar/update-django-nose
• andya/update-requirements
• benp/travis-use-docker
• bmedx/upgrade_django-model-utils
• cliff/incr-109-upgrade-ddt
• cliff/incr-110-upgrade-freezegun
• cliff/incr-111-fix-migrations
• dcs/restframework-version
• diana/add-app-label
• diana/add-version-tagging
• diana/all-submissions
• diana/linting-updates
• diana/remove-datadog
• diana/score-annotations
• diana/update-submissions-version
• diana/update-version
• django-upgrade/1.8
• django1.8-upgrade
• efischer/1_0_0_release
• efischer/EDU-1090
• efischer/annotation_serialization
• efischer/archive_table
• efischer/clear_state
• efischer/delete_studentitem
• efischer/django_unpin
• efischer/django_version
• efischer/gemnasium
• efischer/jsonfield
• efischer/loosen_reqs
• efischer/mangle_key
• efischer/now_where_were_we
• efischer/pytz
• efischer/score_summary
• efischer/send_time
• efischer/slice_all
• efischer/swallow_reset_signal
• efischer/swear_words
• efischer/version_bump
• estute/testing
• estute/version-3.0
• feanil/update_repo_metadata
• jeskew/bug_fix_studentitemserializer
• jeskew/drf_version_bump
• jeskew/further_logging_field_errors
• jeskew/log_more_info_on_error
• jeskew/submissions_version_bump
• jkantor/mgmt
• jkantor/mgmt_ver
• master
• muzaffar/fix-top-submissions-tnl1455
• muzaffar/removed-sub-with-0-score-tnl1125
• muzaffar/upgrage-django-1.8
• nedbat/dont-omit-tests
• neem/score-summaries
• update-pytz
• will/add-migrations-to-setup-py
• will/cache-timeout-top-scores
• will/clarify-travis
• will/docs
• will/optimize-top-scores
• will/read-replica
• will/submission-request-error
• will/upgrade-django-extensions
• will/upgrade-django-model-utils
• will/upgrade-django-rest-framework

Build # 290

Build Type

push

travis-ci

Committed by efischer19

Commit Message

Soft-delete Submissions to reset student state

Previous methods of "deleting state" for a student had just issued
reset scores, so the (unscored) submission was still hanging around.
With this change, the submission can be "deleted" for all django-
relevant purposes, while still remaining in the database in case
it turns out to be interesting to analytics later.

Includes a migration that may take some time to run, and will acquire a
table lock. This will cause an outage for any other request trying to
hit the submissions table during the migration. We've estimated the outage
to last approximately 3 minutes on a database with 700,000 rows.

Test also included.

Run Details

274 of 468 relevant lines covered (58.55%)

0.59 hits per line