From e10d33decae105fd7bbed03c6235d8b2c699265c Mon Sep 17 00:00:00 2001 From: Cathy Teng <70817427+cathteng@users.noreply.github.com> Date: Thu, 7 Sep 2023 11:13:26 -0700 Subject: [PATCH] chore(github-growth): add column for file language in CommitFileChange (#55858) --- migrations_lockfile.txt | 2 +- .../0544_add_commitfilechange_language_col.py | 31 +++++++++++++++++++ src/sentry/models/commitfilechange.py | 1 + 3 files changed, 33 insertions(+), 1 deletion(-) create mode 100644 src/sentry/migrations/0544_add_commitfilechange_language_col.py diff --git a/migrations_lockfile.txt b/migrations_lockfile.txt index a7e88b5baa1f24..6c95a65b37da8d 100644 --- a/migrations_lockfile.txt +++ b/migrations_lockfile.txt @@ -7,5 +7,5 @@ will then be regenerated, and you should be able to merge without conflicts. nodestore: 0002_nodestore_no_dictfield replays: 0003_add_size_to_recording_segment -sentry: 0543_add_team_id_to_groupsubscription +sentry: 0544_add_commitfilechange_language_col social_auth: 0002_default_auto_field diff --git a/src/sentry/migrations/0544_add_commitfilechange_language_col.py b/src/sentry/migrations/0544_add_commitfilechange_language_col.py new file mode 100644 index 00000000000000..2ec7b08088c83c --- /dev/null +++ b/src/sentry/migrations/0544_add_commitfilechange_language_col.py @@ -0,0 +1,31 @@ +# Generated by Django 3.2.20 on 2023-09-07 17:39 + +from django.db import migrations, models + +from sentry.new_migrations.migrations import CheckedMigration + + +class Migration(CheckedMigration): + # This flag is used to mark that a migration shouldn't be automatically run in production. For + # the most part, this should only be used for operations where it's safe to run the migration + # after your code has deployed. So this should not be used for most operations that alter the + # schema of a table. + # Here are some things that make sense to mark as dangerous: + # - Large data migrations. Typically we want these to be run manually by ops so that they can + # be monitored and not block the deploy for a long period of time while they run. + # - Adding indexes to large tables. Since this can take a long time, we'd generally prefer to + # have ops run this and not block the deploy. Note that while adding an index is a schema + # change, it's completely safe to run the operation after the code has deployed. + is_dangerous = False + + dependencies = [ + ("sentry", "0543_add_team_id_to_groupsubscription"), + ] + + operations = [ + migrations.AddField( + model_name="commitfilechange", + name="language", + field=models.CharField(default=None, max_length=255, null=True), + ), + ] diff --git a/src/sentry/models/commitfilechange.py b/src/sentry/models/commitfilechange.py index 7ac9d91ef69006..b227184188e9c2 100644 --- a/src/sentry/models/commitfilechange.py +++ b/src/sentry/models/commitfilechange.py @@ -31,6 +31,7 @@ class CommitFileChange(Model): type = models.CharField( max_length=1, choices=(("A", "Added"), ("D", "Deleted"), ("M", "Modified")) ) + language = models.CharField(default=None, null=True, max_length=255) objects = CommitFileChangeManager()