Francois Gouget : testbot: Require that a Job' s Submitted field always be set.

Alexandre Julliard julliard at winehq.org
Thu Jan 18 09:28:03 CST 2018


Module: tools
Branch: master
Commit: 5ddc51c384124f7c5b3db06e5da02d55c6add7bc
URL:    https://source.winehq.org/git/tools.git/?a=commit;h=5ddc51c384124f7c5b3db06e5da02d55c6add7bc

Author: Francois Gouget <fgouget at codeweavers.com>
Date:   Thu Jan 18 03:58:23 2018 +0100

testbot: Require that a Job's Submitted field always be set.

This guarantees we have at least one field to purge old jobs.

Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 testbot/ddl/update30.sql           | 4 ++++
 testbot/ddl/winetestbot.sql        | 2 +-
 testbot/doc/winetestbot-schema.dia | 2 +-
 3 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/testbot/ddl/update30.sql b/testbot/ddl/update30.sql
new file mode 100644
index 0000000..892d8d1
--- /dev/null
+++ b/testbot/ddl/update30.sql
@@ -0,0 +1,4 @@
+USE winetestbot;
+
+ALTER TABLE Jobs
+  MODIFY Submitted DATETIME NOT NULL;
diff --git a/testbot/ddl/winetestbot.sql b/testbot/ddl/winetestbot.sql
index ad8d1a9..cebc265 100644
--- a/testbot/ddl/winetestbot.sql
+++ b/testbot/ddl/winetestbot.sql
@@ -114,7 +114,7 @@ CREATE TABLE Jobs
   Priority   INT(1)      NOT NULL,
   Status     ENUM('queued', 'running', 'completed', 'badpatch', 'badbuild', 'boterror', 'canceled') NOT NULL,
   Remarks    VARCHAR(128) NULL,
-  Submitted  DATETIME    NULL,
+  Submitted  DATETIME    NOT NULL,
   Ended      DATETIME    NULL,
   PatchId    INT(7)      NULL,
   FOREIGN KEY (BranchName) REFERENCES Branches(Name),
diff --git a/testbot/doc/winetestbot-schema.dia b/testbot/doc/winetestbot-schema.dia
index ef7c3ca..7a94834 100644
--- a/testbot/doc/winetestbot-schema.dia
+++ b/testbot/doc/winetestbot-schema.dia
@@ -1450,7 +1450,7 @@
             <dia:boolean val="false"/>
           </dia:attribute>
           <dia:attribute name="nullable">
-            <dia:boolean val="true"/>
+            <dia:boolean val="false"/>
           </dia:attribute>
           <dia:attribute name="unique">
             <dia:boolean val="false"/>




More information about the wine-cvs mailing list