Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(group-attributes): Add new column for group first release #6070

Merged
merged 1 commit into from
Jun 26, 2024

Conversation

snigdhas
Copy link
Member

Add a new column with a different name for the Group.first_release field to the GroupAttributes table.

Un-revert of #6055

Copy link

This PR has a migration; here is the generated SQL

-- start migrations

-- forward migration group_attributes : 0004_add_new_first_release_column_to_group_attributes
Local op: ALTER TABLE group_attributes_local ADD COLUMN IF NOT EXISTS group_first_release Nullable(UInt64) AFTER group_priority;
Distributed op: ALTER TABLE group_attributes_dist ADD COLUMN IF NOT EXISTS group_first_release Nullable(UInt64) AFTER group_priority;
-- end forward migration group_attributes : 0004_add_new_first_release_column_to_group_attributes




-- backward migration group_attributes : 0004_add_new_first_release_column_to_group_attributes
Distributed op: ALTER TABLE group_attributes_dist DROP COLUMN IF EXISTS group_first_release;
Local op: ALTER TABLE group_attributes_local DROP COLUMN IF EXISTS group_first_release;
-- end backward migration group_attributes : 0004_add_new_first_release_column_to_group_attributes

@snigdhas snigdhas marked this pull request as ready for review June 26, 2024 20:25
@snigdhas snigdhas requested a review from a team as a code owner June 26, 2024 20:25
@volokluev
Copy link
Member

volokluev commented Jun 26, 2024

@snigdhas snigdhas requested review from ayirr7 and a team June 26, 2024 20:36
@snigdhas snigdhas merged commit b7de65e into master Jun 26, 2024
28 checks passed
@snigdhas snigdhas deleted the snigdha/new-column-again branch June 26, 2024 20:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants