The default bug view has changed. See this FAQ.

collectstats.pl --regenerate fails on PostgreSQL

RESOLVED FIXED in Bugzilla 3.6

Status

()

Bugzilla
Reporting/Charting
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: Max Kanat-Alexander, Assigned: Max Kanat-Alexander)

Tracking

Bugzilla 3.6
Bug Flags:
approval +
approval3.6 +
blocking3.6.2 +

Details

Attachments

(1 attachment)

(Assignee)

Description

7 years ago
Doing collectstats.pl --regenerate fails on PostgreSQL (and probably any non-MySQL database) because there's a SQL error that is coincidentally worked around by the DB::Mysql implementation of sql_to_days.
Flags: blocking3.6.2+
(Assignee)

Comment 1

7 years ago
Created attachment 456140 [details] [diff] [review]
v1

This adds a single space, to fix the problem. :-)
Assignee: charting → mkanat
Status: NEW → ASSIGNED
Attachment #456140 - Flags: review?(reed)
Comment on attachment 456140 [details] [diff] [review]
v1

r=me by inspection.

Surprised nobody noticed this sooner, considering it should throw some type of SQL error.
Attachment #456140 - Flags: review?(reed) → review+
Flags: approval?
Flags: approval3.6?

Updated

7 years ago
Flags: approval?
Flags: approval3.6?
Flags: approval3.6+
Flags: approval+
(Assignee)

Comment 3

7 years ago
Yeah, I'm surprised as well. Well, --regenerate isn't that commonly used or known about, I suppose.

In any case, thanks for the review! :-)

Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/trunk/
modified collectstats.pl
Committed revision 7283.

Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/3.6/
modified collectstats.pl
Committed revision 7122.
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.