[Qa-jenkins-scm] [jenkins.debian.net] 01/05: reproducible: db: results.build_duration should be type integer, so ORDER BY works as it should

[Holger Levsen]

This is an automated email from the git hooks/post-receive script.

holger pushed a commit to branch master
in repository jenkins.debian.net.

commit 639abbad1a430cdbe45873ec81e16120ff90bc93
Author: Mattia Rizzolo <mattia at mapreri.org>
Date:   Fri Mar 6 20:17:49 2015 +0100

    reproducible: db: results.build_duration should be type integer, so ORDER BY works as it should
---
 TODO                               |  1 -
 bin/reproducible_db_maintenance.py | 27 +++++++++++++++++++++++++++
 2 files changed, 27 insertions(+), 1 deletion(-)

diff --git a/TODO b/TODO
index e9bdc85..4813ba9 100644
--- a/TODO
+++ b/TODO
@@ -134,7 +134,6 @@ properties:
 ** replace bin/reproducible_installed_on_debian.org with a proper data provider from DSA
 
 * db schema 2.0
-** maybe results.build_duration should be type integer, so ORDER BY works as it should.
 ** add support for the save_artifacts field in the schedule table, so we can manually schedule packages, which provide the binary packages somewhere for inspection
 ** move "untested" field in stats table too? (as in csv output...)
 
diff --git a/bin/reproducible_db_maintenance.py b/bin/reproducible_db_maintenance.py
index 85cd24e..74fd8e9 100755
--- a/bin/reproducible_db_maintenance.py
+++ b/bin/reproducible_db_maintenance.py
@@ -266,6 +266,33 @@ schema_updates = {
         '''DROP TABLE stats_builds_age;''',
         '''ALTER TABLE stats_builds_age_tmp RENAME TO stats_builds_age;''',
         'INSERT INTO rb_schema VALUES ("6", "' + now + '")'],
+    7: [ # change build_duration field in results and stats_build from str to int
+        '''CREATE TABLE stats_build_tmp
+           (id INTEGER PRIMARY KEY,
+            name TEXT NOT NULL,
+            version TEXT NOT NULL,
+            suite TEXT NOT NULL,
+            architecture TEXT NOT NULL,
+            status TEXT NOT NULL,
+            build_date TEXT NOT NULL,
+            build_duration INTEGER NOT NULL,
+            UNIQUE (name, version, suite, architecture, build_date))''',
+        'INSERT INTO stats_build_tmp SELECT * FROM stats_build',
+        'DROP TABLE stats_build',
+        'ALTER TABLE stats_build_tmp RENAME TO stats_build',
+        '''CREATE TABLE results_tmp
+           (id INTEGER PRIMARY KEY,
+            package_id INTEGER NOT NULL,
+            version TEXT NOT NULL,
+            status TEXT,
+            build_date TEXT,
+            build_duration INTEGER DEFAULT '0',
+            UNIQUE (package_id)
+            FOREIGN KEY(package_id) REFERENCES sources(id))''',
+        'INSERT INTO results_tmp SELECT * FROM results',
+        'DROP TABLE results',
+        'ALTER TABLE results_tmp RENAME TO results',
+        'INSERT INTO rb_schema VALUES ("7", "' + now + '")'],
 }
 
 

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/qa/jenkins.debian.net.git