incorrect column definition for bugs.delta_ts

RESOLVED FIXED in Bugzilla 2.18

Status

()

P3
trivial
RESOLVED FIXED
17 years ago
6 years ago

People

(Reporter: myk, Assigned: goobix)

Tracking

2.15
Bugzilla 2.18
Bug Flags:
approval +

Details

Attachments

(1 attachment)

(Reporter)

Description

17 years ago
The bugs.delta_ts field is defined as being able to contain NULL values, even
though it is initialized to the current date/time when a new record is created
in that table and is never set to NULL.  It should be defined as *not* being
able to contain NULL values.
Priority: -- → P3
Target Milestone: --- → Bugzilla 2.18
(Assignee)

Comment 1

15 years ago
Created attachment 131629 [details] [diff] [review]
Patch to make delta_ts not null
(Assignee)

Updated

15 years ago
Attachment #131629 - Flags: review?(kiko)
(Assignee)

Comment 2

15 years ago
This patch does not modify the structure of an existing table, I think it
affects only new tables created by checksetup. Modifying the existing ones too
might be overkill and I don't know if that would be needed as well.
Yeah, we can fix old systems when we change it from TIMESTAMP to DATETIME

Comment 4

15 years ago
Comment on attachment 131629 [details] [diff] [review]
Patch to make delta_ts not null

I agree that it's not important to change existing tables, and justdave
confirmed, so r=kiko.
Attachment #131629 - Flags: review?(kiko) → review+
(Assignee)

Comment 5

15 years ago
Reassigning to patch writer.
Assignee: justdave → jocuri
Flags: approval?
(Assignee)

Updated

15 years ago
Status: NEW → ASSIGNED
Flags: approval? → approval+

Comment 6

15 years ago
Checked in:

/cvsroot/mozilla/webtools/bugzilla/checksetup.pl,v  <--  checksetup.pl
new revision: 1.251; previous revision: 1.250

(And vladd chalks another one off!)
Status: ASSIGNED → RESOLVED
Last Resolved: 15 years ago
Resolution: --- → FIXED
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.