From b90e6289ac4c370e746170a03eb1d1c696103abf Mon Sep 17 00:00:00 2001 From: Christopher Baines Date: Sun, 29 Sep 2019 17:10:04 +0100 Subject: Fix the 'NULL' values in git_branches for the commits The git_branches table had 'NULL' values for some commits where the branch was deleted, importantly this was the string 'NULL', not an actual NULL value. This commit fixes that, migrating the existing values to be '', and changing the relevant code. The primary key is also extended to include the datetime field, as this is important to allow a branch to be deleted twice. --- sqitch/deploy/fix_null_values_in_git_branches.sql | 11 +++++++++++ sqitch/revert/fix_null_values_in_git_branches.sql | 7 +++++++ sqitch/sqitch.plan | 1 + sqitch/verify/fix_null_values_in_git_branches.sql | 7 +++++++ 4 files changed, 26 insertions(+) create mode 100644 sqitch/deploy/fix_null_values_in_git_branches.sql create mode 100644 sqitch/revert/fix_null_values_in_git_branches.sql create mode 100644 sqitch/verify/fix_null_values_in_git_branches.sql (limited to 'sqitch') diff --git a/sqitch/deploy/fix_null_values_in_git_branches.sql b/sqitch/deploy/fix_null_values_in_git_branches.sql new file mode 100644 index 0000000..468d041 --- /dev/null +++ b/sqitch/deploy/fix_null_values_in_git_branches.sql @@ -0,0 +1,11 @@ +-- Deploy guix-data-service:fix_null_values_in_git_branches to pg + +BEGIN; + +ALTER TABLE git_branches DROP CONSTRAINT git_branches_pkey; + +UPDATE git_branches SET commit = '' WHERE commit = 'NULL'; + +ALTER TABLE git_branches ADD PRIMARY KEY (name, commit, git_repository_id, datetime); + +COMMIT; diff --git a/sqitch/revert/fix_null_values_in_git_branches.sql b/sqitch/revert/fix_null_values_in_git_branches.sql new file mode 100644 index 0000000..e4991fc --- /dev/null +++ b/sqitch/revert/fix_null_values_in_git_branches.sql @@ -0,0 +1,7 @@ +-- Revert guix-data-service:fix_null_values_in_git_branches from pg + +BEGIN; + +-- XXX Add DDLs here. + +COMMIT; diff --git a/sqitch/sqitch.plan b/sqitch/sqitch.plan index ff17ebd..859258a 100644 --- a/sqitch/sqitch.plan +++ b/sqitch/sqitch.plan @@ -22,3 +22,4 @@ lint_warnings 2019-08-18T17:10:12Z Christopher Baines # Store guix_revision_lint_checkers 2019-09-01T12:17:38Z chris # Associate lint_checkers to guix_revisions package_versions_by_guix_revision_range 2019-09-26T20:23:15Z Christopher Baines # Add package_versions_by_guix_revision_range table index_on_load_new_guix_revision_job_events_job_id 2019-09-29T10:39:04Z Christopher Baines # Add index on load_new_guix_revision_job_events.job_id +fix_null_values_in_git_branches 2019-09-29T11:06:12Z Christopher Baines # Fix the NULL values in git_branches diff --git a/sqitch/verify/fix_null_values_in_git_branches.sql b/sqitch/verify/fix_null_values_in_git_branches.sql new file mode 100644 index 0000000..b8176a8 --- /dev/null +++ b/sqitch/verify/fix_null_values_in_git_branches.sql @@ -0,0 +1,7 @@ +-- Verify guix-data-service:fix_null_values_in_git_branches on pg + +BEGIN; + +-- XXX Add verifications here. + +ROLLBACK; -- cgit v1.2.3