Skip to content

adjust index length name#690

Open
adrianviquez wants to merge 2 commits intomainfrom
adrian/more-export-performance
Open

adjust index length name#690
adrianviquez wants to merge 2 commits intomainfrom
adrian/more-export-performance

Conversation

@adrianviquez
Copy link
Contributor

@adrianviquez adrianviquez commented Feb 7, 2026

Adjust index for 30 character limit issue

Legal Boilerplate

Look, I get it. The entity doing business as "Sentry" was incorporated in the State of Delaware in 2015 as Functional Software, Inc. In 2022 this entity acquired Codecov and as result Sentry is going to need some rights from me in order to utilize my contributions in this PR. So here's the deal: I retain all rights, title and interest in and to my contributions, and by keeping this boilerplate intact I confirm that Sentry can use, modify, copy, and redistribute my contributions, under Sentry's choice of terms.


Note

Low Risk
Purely renames a database index (no schema/data shape changes), with low blast radius aside from ensuring the migration applies cleanly on existing databases.

Overview
Renames the UploadLevelTotals composite index on (report_session_id, id) from uploadleveltotals_session_id_idx to ult_session_id_idx.

Updates both the Django model Meta.indexes and migration 0045 (including the documented SQL) to use the shorter name, avoiding index-name length issues while keeping the indexed columns unchanged.

Written by Cursor Bugbot for commit 55d995f. This will update automatically on new commits. Configure here.

@adrianviquez adrianviquez requested a review from a team as a code owner February 7, 2026 01:39
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

@adrianviquez adrianviquez disabled auto-merge February 7, 2026 01:42
@sentry
Copy link

sentry bot commented Feb 7, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.34%. Comparing base (304d215) to head (55d995f).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #690   +/-   ##
=======================================
  Coverage   92.34%   92.34%           
=======================================
  Files        1302     1302           
  Lines       47805    47805           
  Branches     1619     1619           
=======================================
  Hits        44146    44146           
  Misses       3350     3350           
  Partials      309      309           
Flag Coverage Δ
sharedintegration 37.12% <ø> (ø)
sharedunit 85.16% <ø> (ø)
workerintegration 58.59% <ø> (ø)
workerunit 90.37% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@codecov-notifications
Copy link

codecov-notifications bot commented Feb 7, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@codspeed-hq
Copy link

codspeed-hq bot commented Feb 7, 2026

CodSpeed Performance Report

Merging this PR will not alter performance

Comparing adrian/more-export-performance (55d995f) with main (304d215)

Summary

✅ 9 untouched benchmarks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants