[elramen]

https://sentry.slack.com/archives/C039ZGT5K/p1792104186179979

Please confirm that it's safe to have a mix of lz4 and zstd mid-transition.

[cursor]

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.}

Want reviews to match your repository better? Bugbot Learning can learn team-specific rules from PR activity. A team admin can enable Learning in the Cursor dashboard.

^{Reviewed by Cursor Bugbot for commit 53539c5. Configure here.}

[cursor]

Rollback does not restore original codec

Low Severity

backwards_ops rebuilds the columns with no codec clause. ClickHouse MODIFY COLUMN that omits CODEC keeps the current codec, so a rollback does not restore the original LZ4 default on attributes_bool and attributes_int.

Additional Locations (1)

• snuba/snuba_migrations/events_analytics_platform/0073_items_bool_int_codecs.py#L29-L35

 

^{Reviewed by Cursor Bugbot for commit 53539c5. Configure here.}