Skip to content

Commit 05cc025

Browse files
committed
Adjust the comment about which reports we skip to match reality.
1 parent 04ffdca commit 05cc025

1 file changed

Lines changed: 3 additions & 4 deletions

File tree

server/reportmanager/management/commands/import_reports_from_bigquery.py

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -34,10 +34,9 @@ def handle(self, *args, **options):
3434

3535
client = bigquery.Client(**params)
3636

37-
# For importing, we ignore reports that have no URL or no comment. In
38-
# theory, a NULL URL shouldn't even be possible, but we have a few
39-
# reports like that. Reports without a comment are very unlikely to be
40-
# actionable.
37+
# For importing, we ignore reports that have a NULL URL or comment.
38+
# These shouldn't even exist, but we have quite a few rows like that
39+
# anyway. Since they're most likely just broken reports, we don't care.
4140
result = client.query_and_wait(
4241
f"""SELECT r.*, t.language_code, t.translated_text
4342
FROM `{settings.BIGQUERY_TABLE}` as r

0 commit comments

Comments
 (0)